]> git.evergreen-ils.org Git - OpenSRF.git/commit
Miscellaneous minor tweaks:
authorscottmk <scottmk@9efc2488-bf62-4759-914b-345cdb29e865>
Mon, 9 Nov 2009 02:47:26 +0000 (02:47 +0000)
committerscottmk <scottmk@9efc2488-bf62-4759-914b-345cdb29e865>
Mon, 9 Nov 2009 02:47:26 +0000 (02:47 +0000)
commit1ed1db1c35c14adb3f19e58a0d7167bad051b697
tree596a5701440ac8f599b5b72f90da1f7a167fecbc
parentd44d23af8fdd73522ca6892830c58d22ed47df2d
Miscellaneous minor tweaks:

1. Moved nested #includes out of the header file and into the
implementation files as needed.

2. Additions and refinements to comments; adjustments to white space.

3. Changed several functions to return void instead of int, since we don't
look at the return values anyway.

4. Added the const qualifier to several function parameters.

5. In osrfRouterHandleAppRequest(): initialize arr[], an array of pointers,
by setting each pointer to NULL.  We had been using memset() on the lot,
relying on a the non-portable assumption that a NULL pointer is
represented by all-bits-zero.

6. Minor rearrangements of the logic here and there, mostly to free things
as soon as we're done with them instead of waiting until the end of the block,
or to defer the declarations of things until we're about to use them.

7. Replaced a couple of calls to jsonObjectToSimpleString() with calls to
jsonObjectGetString(), in order to eliminate a malloc() and a free().

8. Renamed osrfRouterHandleAppResponse() to osrfRouterSendAppResponse(),
which is more descriptive and less vague.

M    src/router/osrf_router.h
M    src/router/osrf_router.c
M    src/router/osrf_router_main.c

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1846 9efc2488-bf62-4759-914b-345cdb29e865
src/router/osrf_router.c
src/router/osrf_router.h
src/router/osrf_router_main.c