]> git.evergreen-ils.org Git - OpenSRF.git/commit
Patch from Scott McKellar:
authormiker <miker@9efc2488-bf62-4759-914b-345cdb29e865>
Mon, 26 Nov 2007 18:20:40 +0000 (18:20 +0000)
committermiker <miker@9efc2488-bf62-4759-914b-345cdb29e865>
Mon, 26 Nov 2007 18:20:40 +0000 (18:20 +0000)
commitcf6337cc25b07c93e17377c75ae669fe87113236
tree19a38ad37243ae65bfe146ba0daf3561586c1dad
parentee6f3815d1552604713ecf4954fb94d8df16a683
Patch from Scott McKellar:

1. Shift toward a camel case style of identifiers.

2. Make functions static when they are not called from elsewhere.

3. Populate all members in newly-allocated structs.

4. Add a couple of consts.

Details:

1. We have two redundant typdefs. I replaced all references to the
lower_case_with_underscores version with references to the camelCase
versions (while keeping the obsolescent typedefs themselves):

   osrf_app_request ==> osrfAppRequest
   osrf_app_session ==> osrfAppSession

2. I deleted the prototypes of two functions that are nowhere
referenced or even defined:

   _osrf_app_session_get_request()
   _osrf_app_session_push_request()

3. I eliminated osrf_app_session_make_locale_req(), replacing it
with the equivalent osrfAppSessionMakeLocaleRequest function.  No
other file references the former.

4. I made the following functions static, and removed their
prototypes from the header, since none is referenced from any other
file:

   osrfAppSessionMakeLocaleRequest()
   osrfAppSessionSendBatch()
   all remaining functions with a leading underscore

5. I explicitly initialized the stateless and session_locale members
of osrfAppSession.

6. I added the const qualifier to a couple of parameters of
osrfAppSessionStatus().

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1144 9efc2488-bf62-4759-914b-345cdb29e865
include/opensrf/osrf_app_session.h
src/libopensrf/osrf_app_session.c