Re: [dev] sbase ls patch

2012-01-23 Thread Connor Lane Smith
Hey, On 22 December 2011 22:35, anonymous wrote: > Tiny cleanup patch.  Now more memory is allocated than necessary. Thanks for the patch, I've applied it to tip. Sorry for the delay, I didn't notice this thread. ;) cls

Re: [dev] sbase ls patch

2011-12-22 Thread anonymous
On Thu, Dec 22, 2011 at 11:10:49PM -, Bjartur Thorlacius wrote: > >Tiny cleanup patch. Now more memory is allocated than necessary. > Is that a good thing or a typo? > Now more memory is allocated than necessary, but after applying this patch it will allocate just as much memory as required.

Re: [dev] sbase ls patch

2011-12-22 Thread Rob
On 22 December 2011 23:10, Bjartur Thorlacius wrote: >> Tiny cleanup patch.  Now more memory is allocated than necessary. > > Is that a good thing or a typo? Looking at the code, it appears to be a typo, now the exact amount of memory is allocated Rob

Re: [dev] sbase ls patch

2011-12-22 Thread Bjartur Thorlacius
Tiny cleanup patch. Now more memory is allocated than necessary. Is that a good thing or a typo?

[dev] sbase ls patch

2011-12-22 Thread anonymous
Tiny cleanup patch. Now more memory is allocated than necessary. diff -r dcb29ce4485f ls.c --- a/ls.c Tue Nov 08 19:35:38 2011 +0100 +++ b/ls.c Fri Dec 23 02:34:01 2011 +0400 @@ -62,7 +62,7 @@ many = (argc > optind+1); if((n = argc - optind) > 0) { - if(!(