On Sunday 13 April 2008 12:20 Scott McKellar wrote:
> This patch tweaks a few things.
>
> 1. In json_parse_json_string() we declare a character array buff[],
> fill it with nuls, and never refer to it again.  I eliminated it.
>
> 2. A few lines below that, we use memset() to clear a three-character
> buffer.  I replaced the memset() with an initializer clause.
>
> 3. in json_handle_error() we were relying on the osrf_clearbuf macro
> to add a terminal nul.  However the plan is for this macro to become
> a no-op someday.  I added the nul manually.

Also applied.  

-bill

-- 
Bill Erickson
| VP, Software Development & Integration
| Equinox Software, Inc. / The Evergreen Experts
| phone: 877-OPEN-ILS (673-6457)
| email: [EMAIL PROTECTED]
| web: http://esilibrary.com

Reply via email to