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

Hi,

I have checked separately so that later if we wish to have separate commands to search above or below separately it would be useful.

Sorry for not following up with the coding standards.

I have updated my version of code and have attached the patch file.

Regards,

Sumedh Pendurkar

On 25 September 2016 at 14:08, Benno Schulenberg <address@hidden> wrote:

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

From:	Sumedh Pendurkar
Subject:	Re: [Nano-devel] Adding of Auto-Complete feature in nano repository
Date:	Tue, 27 Sep 2016 18:56:19 +0530