zsh-workers
 help / color / mirror / code / Atom feed
* [PATCH] Unbalanced parenthesis in _urls
@ 2015-05-20 12:03 İsmail Dönmez
  2015-05-20 12:54 ` Peter Stephenson
  0 siblings, 1 reply; 4+ messages in thread
From: İsmail Dönmez @ 2015-05-20 12:03 UTC (permalink / raw)
  To: Zsh Hackers' List

[-- Attachment #1: Type: text/plain, Size: 61 bytes --]

Another fallout from workers/35168 , patch attached.

ismail

[-- Attachment #2: url-parens.patch --]
[-- Type: text/x-patch, Size: 432 bytes --]

Index: zsh/Completion/Unix/Type/_urls
===================================================================
--- zsh.orig/Completion/Unix/Type/_urls
+++ zsh/Completion/Unix/Type/_urls
@@ -75,7 +75,7 @@ fi
 scheme="$match[1]"
 
 case "$scheme" in
-  http(|s)|(|s)ftp|scp|gopher)
+  (http(|s)|(|s)ftp|scp|gopher)
     if ! compset -P //; then
       _wanted -C "$scheme" prefixes expl 'end of prefix' compadd -S '' "$@" //
       return

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] Unbalanced parenthesis in _urls
  2015-05-20 12:03 [PATCH] Unbalanced parenthesis in _urls İsmail Dönmez
@ 2015-05-20 12:54 ` Peter Stephenson
  2015-05-20 12:57   ` İsmail Dönmez
  0 siblings, 1 reply; 4+ messages in thread
From: Peter Stephenson @ 2015-05-20 12:54 UTC (permalink / raw)
  To: Zsh Hackers' List

On Wed, 20 May 2015 15:03:21 +0300
İsmail Dönmez <ismail@donmez.ws> wrote:
> Another fallout from workers/35168 , patch attached.

I'm happy to apply these if they're causing issues, but the parenthesis
problem will need sorting out properly, so don't bother seeking things
out.

pws


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] Unbalanced parenthesis in _urls
  2015-05-20 12:54 ` Peter Stephenson
@ 2015-05-20 12:57   ` İsmail Dönmez
  2015-05-20 13:07     ` Peter Stephenson
  0 siblings, 1 reply; 4+ messages in thread
From: İsmail Dönmez @ 2015-05-20 12:57 UTC (permalink / raw)
  To: Peter Stephenson; +Cc: Zsh Hackers' List

On Wed, May 20, 2015 at 3:54 PM, Peter Stephenson
<p.stephenson@samsung.com> wrote:
> On Wed, 20 May 2015 15:03:21 +0300
> İsmail Dönmez <ismail@donmez.ws> wrote:
>> Another fallout from workers/35168 , patch attached.
>
> I'm happy to apply these if they're causing issues, but the parenthesis
> problem will need sorting out properly, so don't bother seeking things
> out.

Well rather than seeking out, it hit me with curl completion :-) Feel
free to discard for now.

ismail


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] Unbalanced parenthesis in _urls
  2015-05-20 12:57   ` İsmail Dönmez
@ 2015-05-20 13:07     ` Peter Stephenson
  0 siblings, 0 replies; 4+ messages in thread
From: Peter Stephenson @ 2015-05-20 13:07 UTC (permalink / raw)
  To: Zsh Hackers' List

On Wed, 20 May 2015 15:57:14 +0300
İsmail Dönmez <ismail@donmez.ws> wrote:
> On Wed, May 20, 2015 at 3:54 PM, Peter Stephenson
> <p.stephenson@samsung.com> wrote:
> > On Wed, 20 May 2015 15:03:21 +0300
> > İsmail Dönmez <ismail@donmez.ws> wrote:
> >> Another fallout from workers/35168 , patch attached.
> >
> > I'm happy to apply these if they're causing issues, but the parenthesis
> > problem will need sorting out properly, so don't bother seeking things
> > out.
> 
> Well rather than seeking out, it hit me with curl completion :-) Feel
> free to discard for now.

I've committed it --- please feel free to send a patche if it hits you
again.

pws


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2015-05-20 13:08 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-05-20 12:03 [PATCH] Unbalanced parenthesis in _urls İsmail Dönmez
2015-05-20 12:54 ` Peter Stephenson
2015-05-20 12:57   ` İsmail Dönmez
2015-05-20 13:07     ` Peter Stephenson

Code repositories for project(s) associated with this public inbox

	https://git.vuxu.org/mirror/zsh/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).