nano-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Nano-devel] Nano with Line Numbering


From: Faissal Bensefia
Subject: Re: [Nano-devel] Nano with Line Numbering
Date: Tue, 06 Sep 2016 21:12:31 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0


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

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


reply via email to

[Prev in Thread] Current Thread [Next in Thread]