Re: [PATCH 1/6] Add memory pool library
- From
Ramkumar Ramachandra <artagnon@gmail.com>
- Date
- Jun 7, 2010, 14:35 UTC
- Message-ID
- <AANLkTilorbTSy0Va7wX2BauiiVlAEUdzDi87DNF-SGNP@mail.gmail.com>
- In-Reply-To
- <AANLkTimd_l0uKHTqgS211fv30hxd-k0pR55YoY4xCua1@mail.gmail.com>
Hi Erik,
Erik Faye-Lund wrote:
Show 5 quoted lines
> On Mon, Jun 7, 2010 at 3:28 PM, Ramkumar Ramachandra <artagnon@gmail.com> wrote: > Adding a win32-implementation of sysconf for this purpose isn't > tricky, but perhaps it's a little bit overkill. Can't we just use some > large power of two number? Hopefully, we could chose a size that is a > multiple of the page-size on all systems...
Until an alternative is ready, I'll just use a large power of two and roll out another revision.
> Anyway, this is roughly what a Windows-implementation of sysconf would > look like (not tested - only checked that it compiles):
Thanks for implementing this. Unfortunately, I don't have a Windows box to test this- why don't you send a patch to the list and sign it off? While I agree that just implementing sysconf for _SC_PAGESIZE is an overkill, others can test and extend the code.
-- Ram