Re: PATCH: some more cleanup

[Sven Wischnowsky <wischnow@xxxxxxxxxxxxxxxxxxxxxxx>]

Bart Schaefer wrote:

> On Nov 16,  1:56pm, Sven Wischnowsky wrote:
> } Subject: PATCH: some more cleanup
> }
> } diff -u -r oldcompletion/Builtins/_stat Completion/Builtins/_stat
> } --- oldcompletion/Builtins/_stat	Tue Nov 16 10:08:17 1999
> } +++ Completion/Builtins/_stat	Tue Nov 16 13:32:47 1999
> } @@ -9,7 +9,7 @@
> }  
> }    while _tags; do
> }      _requested options expl 'inode element' &&
> } -        { ! _style elements prefix-needed || [[ "$PREFIX[1]" = + ]] } &&
> } +        { ! _style options prefix-needed || [[ "$PREFIX[1]" = + ]] } &&
> }          compadd "$expl[@]" - +device +inode +mode +nlink +uid +gid +rdev \
> }                               +size +atime +mtime +ctime +blksize +block +link
> }      fi
> 
> This hunk failed for me.  Here's a fragment of what I have; I see two
> different lines, not one:
> 
>     _requested elements expl 'inode element' &&
>                ^^^^^^^^
>         { ! _style elements prefix-needed || [[ "$PREFIX[1]" = + ]] } &&
>                    ^^^^^^^^
> 
> To apply by hand, just change both "elements" to "options".

And in another message:

> Hmm, I just spotted a *third* place where "elements" should be changed to
> "options" in Completion/Builtins/_stat:
> 
>   _tags elements files || return 1
>         ^^^^^^^^

Both of these `elements' were changed by 8647.

Bye
 Sven


--
Sven Wischnowsky                         wischnow@xxxxxxxxxxxxxxxxxxxxxxx