On Tue, 18 Nov 2008 19:47:29 -0500, Scott McKellar <[EMAIL PROTECTED]> wrote:

--- On Tue, 11/18/08, Bill Erickson <[EMAIL PROTECTED]> wrote:

OK, I tracked down 2 problems, the first makes sense, the
second, not so
much.

The first problem was the result of attempting to escape
quotes (") and
backslashes (\) in the final if/else block of
buffer_append_uescape().
Since " and \ are both >= ' ', they
were not getting escaped.

Doh! <blush/> Sorry.

The second was caused by NULL jsonObject's being passed
from
OSRF_LIST_GET_INDEX (osrf_json_object:337) into
add_json_to_buffer().
osrfList allows NULL entries, so it seems the old could
should have been
dying here, but for some reason only the new code suffers.

This one I don't understand.  Is it working now, somehow?  If it is,
then I won't worry about it.


Yep, by all accounts, OpenSRF/Evergreen are happy with the changes.

-b

--
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