Re: [9fans] test(1) -older bug?

2013-06-03 Thread Richard Miller
its /n/sources/patch/testolder, also leaks dir in the case: if(rel) n = time(0) - n; if(n 0) return 0; - HERE r = dir-mtime n; free(dir); return r; And the consequences of not freeing a few bytes of memory, in a command which

Re: [9fans] test(1) -older bug?

2013-06-03 Thread Jacob Todd
On Jun 3, 2013 6:49 AM, Richard Miller 9f...@hamnavoe.com wrote: And the consequences of not freeing a few bytes of memory, in a command which will exit a few microseconds later, would be ... ? bad taste.

Re: [9fans] test(1) -older bug?

2013-06-03 Thread Richard Miller
bad taste. I agree - better style is to free memory always, or never. But the choice might not be as obvious as one first thinks.

Re: [9fans] test(1) -older bug?

2013-06-03 Thread arnold
Richard Miller 9f...@hamnavoe.com wrote: And the consequences of not freeing a few bytes of memory, in a command which will exit a few microseconds later, would be ... ? The Code Correctness Police come and collect you and force you to program on Windows... ducks :-) Arnold

Re: [9fans] test(1) -older bug?

2013-06-03 Thread erik quanstrom
On Mon Jun 3 08:00:20 EDT 2013, arn...@skeeve.com wrote: Richard Miller 9f...@hamnavoe.com wrote: And the consequences of not freeing a few bytes of memory, in a command which will exit a few microseconds later, would be ... ? The Code Correctness Police come and collect you and force

Re: [9fans] test(1) -older bug?

2013-06-02 Thread cinap_lenrek
its /n/sources/patch/testolder, also leaks dir in the case: if(rel) n = time(0) - n; if(n 0) return 0; - HERE r = dir-mtime n; free(dir); return r; -- cinap

Re: [9fans] test(1) -older bug?

2013-06-02 Thread erik quanstrom
On Sun Jun 2 17:31:05 EDT 2013, cinap_len...@gmx.de wrote: its /n/sources/patch/testolder, also leaks dir in the case: if(rel) n = time(0) - n; if(n 0) return 0; - HERE r = dir-mtime n; free(dir); return r; thanks. i

Re: [9fans] test(1) -older bug?

2013-06-01 Thread erik quanstrom
yup. i think it's a bug: /n/sources/patch/older /n/atom/patch/older - erik