Re: [Nano-devel] Adding of Auto-Complete feature in nano repository

[Benno Schulenberg]

On Thu, Sep 22, 2016, at 12:34, Sumedh Pendurkar wrote:
> I have been using nano for a long time now, Often while writing programs,
> typing the variable names was troublesome. So i have modified the source
> code of nano-2.6.2 and I have added a shortcut that would look up  for the
> current word in the current file and replace it if only match is found or
> suggest words if more than one word is found.

Interesting feature.  Thanks for the patch.  I don't have time to
look at this in detail for some weeks, but after a quick look: why
separately look for matches before and after the cursor position?
Why not run through the entire file from top to bottom?  That is:
from 'fileage' to 'filebot'.  Or do you wish to have the closer
matches to take precedence?

Some details: please use a blank line between functions, and give
each function an introductory comment.  Always use a space after
'if' and 'for' and comma.  And elide check_if_exceeds().

Benno

-- 
http://www.fastmail.com - Faster than the air-speed velocity of an
                          unladen european swallow