From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 1162 invoked by alias); 9 Feb 2015 14:01:45 -0000 Mailing-List: contact zsh-workers-help@zsh.org; run by ezmlm Precedence: bulk X-No-Archive: yes List-Id: Zsh Workers List List-Post: List-Help: X-Seq: 34482 Received: (qmail 3677 invoked from network); 9 Feb 2015 14:01:42 -0000 X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) 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,RCVD_IN_DNSWL_LOW autolearn=ham version=3.3.2 DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d= daniel.shahaf.name; h=x-sasl-enc:date:from:to:subject:message-id :references:mime-version:content-type:in-reply-to; s=mesmtp; bh= EE/kU1GM7qCzT9u+raBW2gNTTso=; b=RZaRqKXEWT/1WF6Xdn37ihiqCJsBKcRZ X9koaHjqUlOkxdh2HYmRfcLPp+LXIesscHyksrpJ0AVtLWpCBxJxsuEKNk2k/tcz Cq7CCIbQopsl9CsxtvVKs/Vy8NKlNgMrSPfoMKiyquSAvwFs6tx/xZayteB2cfsK h5CABcmWnWs= DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d= messagingengine.com; h=x-sasl-enc:date:from:to:subject :message-id:references:mime-version:content-type:in-reply-to; s= smtpout; bh=EE/kU1GM7qCzT9u+raBW2gNTTso=; b=qQe5fhOPcwkNimoDgs9q hZl/ScrAGuEmLSyIV4UcCNqPvj+1C09sHcAlDZF8uEIqdoeDIJkW+WgV06p6qD/0 yJJwmBDpE9Lx9R9sfnGcTx0HQ8Qtp30vpz/Dqf9N6aQXV2dVOcWiKZEaXtk0oWLM jGcp7zIF67ywAp1699gROtw= X-Sasl-enc: Nfm1CQ+In3+n/puBS7V1N1Mu9mcEBDizMxVfBUJ/AlEf 1423490498 Date: Mon, 9 Feb 2015 14:01:24 +0000 From: Daniel Shahaf To: zsh-workers@zsh.org Subject: Re: [PATCH] Update mtr completion to mtr 0.82 Message-ID: <20150209140114.GA1833@tarsus.local2> References: <20150207221226.GA2083@tarsus.local2> <150207162843.ZM10863@torch.brasslantern.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <150207162843.ZM10863@torch.brasslantern.com> User-Agent: Mutt/1.5.21 (2010-09-15) Bart Schaefer wrote on Sat, Feb 07, 2015 at 16:28:43 -0800: > On Feb 7, 10:12pm, Daniel Shahaf wrote: > } > } The -s/-p options are switched > > Has the completion function always had this wrong, or did mtr actually > swap the meaning of these options in a new revision? Just curious. I checked; it seems mtr changed the meaning: % cd mtr-0.85 % grep psize\" -R . ./mtr.c: { "psize", 1, 0, 's' }, /* changed 'p' to 's' to match ping option