From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <zsh-workers-return-29745-mason-zsh=primenet.com.au@zsh.org>
Received: (qmail 2953 invoked by alias); 30 Aug 2011 15:23:29 -0000
Mailing-List: contact zsh-workers-help@zsh.org; run by ezmlm
Precedence: bulk
X-No-Archive: yes
List-Id: Zsh Workers List <zsh-workers.zsh.org>
List-Post: <mailto:zsh-workers@zsh.org>
List-Help: <mailto:zsh-workers-help@zsh.org>
X-Seq: 29745
Received: (qmail 24643 invoked from network); 30 Aug 2011 15:23:28 -0000
X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on f.primenet.com.au
X-Spam-Level: 
X-Spam-Status: No, score=-2.7 required=5.0 tests=BAYES_00,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,
	T_TO_NO_BRKTS_FREEMAIL autolearn=ham version=3.3.1
Received-SPF: pass (ns1.primenet.com.au: SPF record at _spf.google.com designates 209.85.212.43 as permitted sender)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
        d=gmail.com; s=gamma;
        h=mime-version:in-reply-to:references:date:message-id:subject:from:to
         :content-type;
        bh=lktw/v9VFpdayhcRGI/4DD5KV8zfwaKCeQa9ct7P64Y=;
        b=Mq8s/wO40SUkU2l9qAD6XJgxdOMjqOVO992ER5oonf7rs0Pn6qrXlt59wA4C9lO1rj
         teuBh/CAwrcEvkts9pGaL/FDKr3ka8Ud+2ulIW3PiIcEB1WQl3GYBT8tDznCHMlQ/D3J
         6Zcp12he9KxnYz3pCJjcOLtanBJUEjtIlJ0RA=
MIME-Version: 1.0
In-Reply-To: <20110825045107.3CC5540CE6@reinforce.localdomain>
References: <20110825045107.3CC5540CE6@reinforce.localdomain>
Date: Tue, 30 Aug 2011 17:23:21 +0200
Message-ID: <CAHYJk3Q=7R_1kCDOcSresejgLga3YBTyduZSJAA2=6v=-dEo1w@mail.gmail.com>
Subject: Re: [PATCH] Escape colon at mplayer -ss option
From: Mikael Magnusson <mikachu@gmail.com>
To: zsh-workers@zsh.org
Content-Type: text/plain; charset=UTF-8

On 25 August 2011 06:42, eagletmt <eagletmt@gmail.com> wrote:
> ---
>  Completion/X/Command/_mplayer |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/Completion/X/Command/_mplayer b/Completion/X/Command/_mplayer
> index 665f44e..3cb10d0 100644
> --- a/Completion/X/Command/_mplayer
> +++ b/Completion/X/Command/_mplayer
> @@ -77,7 +77,7 @@ _x_arguments -C -s \
>   '-sid[turn on DVD subtitles]:language id' \
>   '-speed[set playback speed rate]:playback speed rate' \
>   '-srate[specify frequency of audio playback]:frequency (Hz)' \
> -  '-ss[seek to given time position]:position (hh:mm\[\:ss\])' \
> +  '-ss[seek to given time position]:position (hh\:mm\[\:ss\])' \
>   '-sstep[specify time between displayed frames]:time (seconds)' \
>   -ssf:mode -stop_xscreensaver \
>   '-stereo:mode:((0\:stereo 1\:left\ channel 2\:right\ channel))' \

rudi_s mentioned on irc that the [] shouldn't be escaped, then I
noticed they're in the wrong place too, so I'll commit this too,

-  '-ss[seek to given time position]:position (hh\:mm\[\:ss\])' \
+  '-ss[seek to given time position]:position ([[hh\:]mm\:]ss[.sss])' \

-- 
Mikael Magnusson