Re: [PATCH] Update feh completion for feh 1.6

[Simon Ruderich <simon@xxxxxxxxxxxx>]

On Tue, Jun 01, 2010 at 05:51:55PM +0200, Daniel Friesel wrote:
> Heya,
>
> I updated the feh completion, it'd be nice if someone could
> commit that to the repo.
>
> --derf

> [snip]
>
>  for arg in ${(k)argument_pairs}; {
> -	arguments+='(--'$arg')-'${argument_pairs[$arg]}${argument_postfix[$arg]}
> +	arguments+='(--'${arg}')-'${argument_pairs[$arg]}${argument_postfix[$arg]}
>  	arguments+='(-'${argument_pairs[$arg]}[1]')--'${arg}${argument_postfix[$arg]}
>  }
>
> [snip]

Looks like this wasn't committed correctly. Here is a patch which
should fix it.

--- a/Completion/Unix/Command/_feh
+++ b/Completion/Unix/Command/_feh
@@ -164,7 +164,8 @@ arguments=(
 )

 for arg in ${(k)argument_pairs}; {
-	arguments+='(--'${arg}')-'${argument_pairs[$arg]}${argument_postfix[$arg	arguments+='(-'${argument_pairs[$arg]}[1]')--'${arg}${argument_postfix[$arg]}
+	arguments+='(--'${arg}')-'${argument_pairs[$arg]}${argument_postfix[$arg]}
+	arguments+='(-'${argument_pairs[$arg]}[1]')--'${arg}${argument_postfix[$arg]}
 }

 function _feh_theme {

Thanks,
Simon
-- 
+ privacy is necessary
+ using gnupg http://gnupg.org
+ public key id: 0x92FEFDB7E44C32F9

Attachment: pgpijPCYp9zIP.pgp
Description: PGP signature