Re: [patch] Remove superfluous seek_filesize() from less

2017-10-28 Thread Ingo Schwarze
Hi, On Sat, Sep 16, 2017 at 10:53:47PM +0200, Jesper Wallin wrote: > I was reading through the code for less/filename.c and noticed that the > calling for seek_filesize() in filesize() is superfluous? A wild guess > is that it's remains from when BAD_LSEEK was removed? There is more that is use

Re: [patch] Remove superfluous seek_filesize() from less

2017-10-05 Thread Jesper Wallin
...and a third bump, this time we're actually out of beta for real. ;-) On Mon, Sep 25, 2017 at 10:19:16AM +0200, Jesper Wallin wrote: > Shameless bump, now when we're out of beta. :-) > > On Sat, Sep 16, 2017 at 10:53:47PM +0200, Jesper Wallin wrote: > > Hi all, > > > > I was reading through th

Re: [patch] Remove superfluous seek_filesize() from less

2017-09-25 Thread Jesper Wallin
On Mon, Sep 25, 2017 at 10:36:18AM +0200, Landry Breuil wrote: > On Mon, Sep 25, 2017 at 10:19:16AM +0200, Jesper Wallin wrote: > > Shameless bump, now when we're out of beta. :-) > > being out of beta means we're in release mode.. Oh! Of course, silly me. Apologize for the noise.

Re: [patch] Remove superfluous seek_filesize() from less

2017-09-25 Thread Landry Breuil
On Mon, Sep 25, 2017 at 10:19:16AM +0200, Jesper Wallin wrote: > Shameless bump, now when we're out of beta. :-) being out of beta means we're in release mode..

Re: [patch] Remove superfluous seek_filesize() from less

2017-09-25 Thread Jesper Wallin
Shameless bump, now when we're out of beta. :-) On Sat, Sep 16, 2017 at 10:53:47PM +0200, Jesper Wallin wrote: > Hi all, > > I was reading through the code for less/filename.c and noticed that the > calling for seek_filesize() in filesize() is superfluous? A wild guess > is that it's remains fro

[patch] Remove superfluous seek_filesize() from less

2017-09-16 Thread Jesper Wallin
Hi all, I was reading through the code for less/filename.c and noticed that the calling for seek_filesize() in filesize() is superfluous? A wild guess is that it's remains from when BAD_LSEEK was removed? Jesper Wallin Index: usr.bin/less/filename.c ===