Re: PATCH: add more malloc.conf details to malloc.3

2014-07-11 Thread Otto Moerbeek
On Fri, Jul 11, 2014 at 01:38:23AM -0600, Theo de Raadt wrote: > I am a bit concerned. The description is trying too hard to be overly > precise, and may be cumbersome for the typical reader of this page. indeed, and I also dislike putting too much implementation details into a man page. Things

Re: PATCH: add more malloc.conf details to malloc.3

2014-07-11 Thread Jason McIntyre
On Fri, Jul 11, 2014 at 07:25:06AM +, Doug Hogan wrote: > On Fri, Jul 11, 2014 at 01:03:54AM -0600, Anthony J. Bentley wrote: > > In addition to what jmc said, if you document these flags they need to > > be marked up with the Cm macro instead of ASCII single quotes. > > My mistake. Fixed ver

Re: PATCH: add more malloc.conf details to malloc.3

2014-07-11 Thread Theo de Raadt
I am a bit concerned. The description is trying too hard to be overly precise, and may be cumbersome for the typical reader of this page. > Index: lib/libc/stdlib/malloc.3 > === > RCS file: /cvs/src/lib/libc/stdlib/malloc.3,v > retri

Re: PATCH: add more malloc.conf details to malloc.3

2014-07-11 Thread Doug Hogan
On Fri, Jul 11, 2014 at 01:03:54AM -0600, Anthony J. Bentley wrote: > In addition to what jmc said, if you document these flags they need to > be marked up with the Cm macro instead of ASCII single quotes. My mistake. Fixed version: Index: lib/libc/stdlib/malloc.3 ==

Re: PATCH: add more malloc.conf details to malloc.3

2014-07-11 Thread Anthony J. Bentley
On Fri, Jul 11, 2014 at 12:50 AM, Jason McIntyre wrote: > On Fri, Jul 11, 2014 at 05:46:15AM +, Doug Hogan wrote: >> .It Cm j >> .Dq Don't Junk . >> By default, small chunks are always junked, and the first part of pages >> @@ -297,6 +303,7 @@ This can substantially aid in compacting >> .\

Re: PATCH: add more malloc.conf details to malloc.3

2014-07-10 Thread Jason McIntyre
On Fri, Jul 11, 2014 at 05:46:15AM +, Doug Hogan wrote: > I don't think the current man page has enough detail for what the > malloc.conf settings do. > > > Index: lib/libc/stdlib/malloc.3 > === > RCS file: /cvs/src/lib/libc/stdl

PATCH: add more malloc.conf details to malloc.3

2014-07-10 Thread Doug Hogan
I don't think the current man page has enough detail for what the malloc.conf settings do. Index: lib/libc/stdlib/malloc.3 === RCS file: /cvs/src/lib/libc/stdlib/malloc.3,v retrieving revision 1.78 diff -u -p -d -r1.78 malloc.3 --- l