[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Nano-devel] [PATCH 1/2 v6] new feature: an option to make the 'next
From: |
Benno Schulenberg |
Subject: |
Re: [Nano-devel] [PATCH 1/2 v6] new feature: an option to make the 'nextword' function stop at word ends |
Date: |
Thu, 10 May 2018 11:35:55 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 |
Op 10-05-18 om 04:42 schreef David Ramsey:
> I've tested this patch out, and it is useful.
Thanks for confirming.
> However, there's one
> oversight in it that I've found.
>
> The new option is supposed to use --afterends as the long option, and -y
> as the short option. However, the latter doesn't actually work, because
> the getopt_long() call in nano.c:main() still uses the unchanged string
> of
>
> "ABC:DEFGHIKLMNOPQ:RST:UVWX:Y:abcdefghijklmno:pqr:s:tuvwxz$"
>
> when it should use
>
> "ABC:DEFGHIKLMNOPQ:RST:UVWX:Y:abcdefghijklmno:pqr:s:tuvwxyz$"
>
> instead.
Oof! Good catch. Thanks. Fixed in the upcoming push.
Benno
signature.asc
Description: OpenPGP digital signature