Re: [Nano-devel] Nano with Line Numbering

[Faissal Bensefia]

On 06/09/16 20:39, Benno Schulenberg wrote:
> 
> On Tue, Sep 6, 2016, at 13:53, Faissal Bensefia wrote:
>> On 28/08/16 09:24, Benno Schulenberg wrote:
>>> About the patch itself.  There is a shorter way to implement
>>> the function intlen().  See the function digits() here:
>>>
>>>     git log -p -1 f111c0 src/utils.c
>>
>> That method is slow, see:
>> http://stackoverflow.com/a/3069580/5269447
> 
> Sure, it is slow.  But you should need it just a few times,
> not for every line -- it should be possible to print the number
> right-aligned in the available space.
> 
I need it for every line drawn so I know how far left to shift the
number when I draw it. Even if I am using it just a few times, I don't
see why I should intentionally use a slower method.

>> I've attached the diffs with git format-patch like you asked.
> 
> Thanks.  It was interesting to see the way you proceed.
> But as an end result I want to see a single patch, based
> off the current state of nano's git.  (All the revision.h
> stuff should be absent from it, for example.  And all the
> reformatting too.  Also, it seems you have some trailing
> whitespace here and there.  Get rid of that too.)
> 
> I haven't tried your changes -- I'll wait for the single patch.
> 
> Benno
> 
I've attached the patch

~Faissal Bensefia

0001-Added-line-number-support.patch
Description: Text Data