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