Re: [OPEN-ILS-DEV] new OpenSRF perl dependency (in trunk)

2008-01-06 Thread Bill Erickson
Mike Rylander wrote: On Jan 3, 2008 8:16 PM, John Schmidt [EMAIL PROTECTED] wrote: On Thursday 03 January 2008, Bill Erickson wrote: Just what everyone wanted for the holidays, right? ;) Well, we've finally found a good Unicode-compliant C-based Perl JSON module called JSON::XS.

[OPEN-ILS-DEV] PATCH: osrf_prefork.[ch] (miscellaneous)

2008-01-06 Thread Scott McKellar
These patches are a code cleanup, and should not affect functionality. 1. I moved almost everything from the header file into the implementation file, since it isn't referenced elsewhere. All that's left is one function prototype and a series of nested #includes. 2. I added compilation guards

Re: [OPEN-ILS-DEV] PATCH: osrf_hash.[ch] (tidying up)

2008-01-06 Thread Mike Rylander
I'm running through this weeks patches now ... so ... On Dec 31, 2007 12:30 AM, Scott McKellar [EMAIL PROTECTED] wrote: These patches: are applied. Thanks, as always! 1. Move several internal details from the header to the implementation file: The macros OSRF_HASH_LIST_SIZE and

Re: [OPEN-ILS-DEV] PATCH: srfsh.c (memory leak)

2008-01-06 Thread Mike Rylander
On Jan 1, 2008 9:17 PM, Scott McKellar [EMAIL PROTECTED] wrote: This leak plugs a memory leak. Applied. In handle_request() we allocate a growing_buffer and pass the pointer to send_request(). However we weren't freeing the growing_buffer after the return from send_request(). Scott