[OPEN-ILS-DEV] PATCH: osrf_legacy_json.c (miscellaneous)
Bill Erickson
erickson at esilibrary.com
Thu May 15 17:25:42 EDT 2008
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: erickson at esilibrary.com
| web: http://esilibrary.com
More information about the Open-ils-dev
mailing list