From: scottmk <scottmk@dcc99617-32d9-48b4-a31d-7c20da2025e4>
Date: Tue, 6 Jan 2009 16:50:36 +0000 (+0000)
Subject: Replaced all occurrences of buffer_data() with the OSRF_BUFFER_C_STR
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=878f7de2afae5c7e9fdd8cf0dc0d988d72af1c64;p=Evergreen.git

Replaced all occurrences of buffer_data() with the OSRF_BUFFER_C_STR
macro, in order to eliminate some mallocs and frees.


git-svn-id: svn://svn.open-ils.org/ILS/trunk@11752 dcc99617-32d9-48b4-a31d-7c20da2025e4
---

diff --git a/Open-ILS/src/c-apps/oils_cstore.c b/Open-ILS/src/c-apps/oils_cstore.c
index 03497f5e5d..b7c7b38e7d 100644
--- a/Open-ILS/src/c-apps/oils_cstore.c
+++ b/Open-ILS/src/c-apps/oils_cstore.c
@@ -110,52 +110,44 @@ int osrfAppInitialize() {
 
     if (!oilsIDLInit( osrf_settings_host_value("/IDL") )) return 1; /* return non-zero to indicate error */
 
-    char* method_str = NULL;
     growing_buffer* method_name = buffer_init(64);
 #ifndef PCRUD
     // Generic search thingy
     buffer_fadd(method_name, "%s.json_query", MODULENAME);
-    method_str = buffer_data(method_name);
-    osrfAppRegisterMethod( MODULENAME, method_str, "doJSONSearch", "", 1, OSRF_METHOD_STREAMING );
-    free(method_str);
+	osrfAppRegisterMethod( MODULENAME, OSRF_BUFFER_C_STR(method_name),
+						   "doJSONSearch", "", 1, OSRF_METHOD_STREAMING );
 #endif
 
     // first we register all the transaction and savepoint methods
     buffer_reset(method_name);
     buffer_fadd(method_name, "%s.transaction.begin", MODULENAME);
-    method_str = buffer_data(method_name);
-    osrfAppRegisterMethod( MODULENAME, method_str, "beginTransaction", "", 0, 0 );
-    free(method_str);
+	osrfAppRegisterMethod( MODULENAME, OSRF_BUFFER_C_STR(method_name),
+						   "beginTransaction", "", 0, 0 );
 
     buffer_reset(method_name);
     buffer_fadd(method_name, "%s.transaction.commit", MODULENAME);
-    method_str = buffer_data(method_name);
-    osrfAppRegisterMethod( MODULENAME, method_str, "commitTransaction", "", 0, 0 );
-    free(method_str);
+	osrfAppRegisterMethod( MODULENAME, OSRF_BUFFER_C_STR(method_name),
+						   "commitTransaction", "", 0, 0 );
 
     buffer_reset(method_name);
     buffer_fadd(method_name, "%s.transaction.rollback", MODULENAME);
-    method_str = buffer_data(method_name);
-    osrfAppRegisterMethod( MODULENAME, method_str, "rollbackTransaction", "", 0, 0 );
-    free(method_str);
+	osrfAppRegisterMethod( MODULENAME, OSRF_BUFFER_C_STR(method_name),
+						   "rollbackTransaction", "", 0, 0 );
 
     buffer_reset(method_name);
     buffer_fadd(method_name, "%s.savepoint.set", MODULENAME);
-    method_str = buffer_data(method_name);
-    osrfAppRegisterMethod( MODULENAME, method_str, "setSavepoint", "", 1, 0 );
-    free(method_str);
+	osrfAppRegisterMethod( MODULENAME, OSRF_BUFFER_C_STR(method_name),
+						   "setSavepoint", "", 1, 0 );
 
     buffer_reset(method_name);
     buffer_fadd(method_name, "%s.savepoint.release", MODULENAME);
-    method_str = buffer_data(method_name);
-    osrfAppRegisterMethod( MODULENAME, method_str, "releaseSavepoint", "", 1, 0 );
-    free(method_str);
+	osrfAppRegisterMethod( MODULENAME, OSRF_BUFFER_C_STR(method_name),
+						   "releaseSavepoint", "", 1, 0 );
 
     buffer_reset(method_name);
     buffer_fadd(method_name, "%s.savepoint.rollback", MODULENAME);
-    method_str = buffer_data(method_name);
-    osrfAppRegisterMethod( MODULENAME, method_str, "rollbackSavepoint", "", 1, 0 );
-    free(method_str);
+	osrfAppRegisterMethod( MODULENAME, OSRF_BUFFER_C_STR(method_name),
+						   "rollbackSavepoint", "", 1, 0 );
 
     buffer_free(method_name);
 
@@ -2890,9 +2882,8 @@ static char* buildSELECT ( jsonObject* search_hash, jsonObject* order_hash, osrf
 		free(join_clause);
 	}
 
-	char* tmpsql = buffer_data(sql_buf); // This strdup's ... no worries.
-	osrfLogDebug(OSRF_LOG_MARK, "%s pre-predicate SQL =  %s", MODULENAME, tmpsql);
-	free(tmpsql);
+	osrfLogDebug(OSRF_LOG_MARK, "%s pre-predicate SQL =  %s",
+				 MODULENAME, OSRF_BUFFER_C_STR(sql_buf));
 
 	buffer_add(sql_buf, " WHERE ");