From be645751d28ffee80864fece75a52f78b6779e85 Mon Sep 17 00:00:00 2001
From: erickson <erickson@9efc2488-bf62-4759-914b-345cdb29e865>
Date: Tue, 13 Jan 2009 18:07:23 +0000
Subject: [PATCH] rolling back latest change because of macro variable
 oddities.  this macro will eventually be replaced with a real function anyway

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1619 9efc2488-bf62-4759-914b-345cdb29e865
---
 include/opensrf/osrf_application.h | 64 ++++++++++--------------------
 1 file changed, 21 insertions(+), 43 deletions(-)

diff --git a/include/opensrf/osrf_application.h b/include/opensrf/osrf_application.h
index 52fdccf..8b1e247 100644
--- a/include/opensrf/osrf_application.h
+++ b/include/opensrf/osrf_application.h
@@ -29,50 +29,28 @@ extern "C" {
 
 /** 
   This macro verifies methods receive the correct parameters */
-#define OSRF_METHOD_VERIFY_CONTEXT_(x) \
-	do { \
-		osrfMethodContext* d = x; \
-		if(!d) return -1; \
-		if(!d->session) { \
-			osrfLogError( OSRF_LOG_MARK, "Session is NULL in app reqeust" ); \
-			return -1; \
-		} \
-		if(!d->method) { \
-			osrfLogError( OSRF_LOG_MARK,  "Method is NULL in app reqeust" ); \
-			return -1; \
-		}\
-		if(d->method->argc) { \
-			if(!d->params) { \
-				osrfLogError( OSRF_LOG_MARK, \
-					"Params is NULL in app reqeust %s", d->method->name ); \
-				return -1; \
-			} \
-			if( d->params->type != JSON_ARRAY ) { \
-				osrfLogError( OSRF_LOG_MARK, \
-					"'params' is not a JSON array for method %s", d->method->name);\
-				return -1; }\
-		}\
-		if( !d->method->name ) { \
-			osrfLogError( OSRF_LOG_MARK, "Method name is NULL"); \
-			return -1; \
-		} \
-	} while(0)
-
-#ifdef OSRF_LOG_PARAMS
-#define OSRF_METHOD_VERIFY_CONTEXT(x) \
- 	do { \
- 		osrfMethodContext* d = x; \
-		OSRF_METHOD_VERIFY_CONTEXT_(d); \
-		char* _j = jsonObjectToJSON(d->params); \
-		if(_j) { \
-			osrfLogInfo( OSRF_LOG_MARK, \
-				"CALL:	%s %s - %s", d->session->remote_service, d->method->name, _j); \
-			free(_j); \
-		} \
-	} \
-	while( 0 )
+#define _OSRF_METHOD_VERIFY_CONTEXT(d) \
+	if(!d) return -1; \
+	if(!d->session) { osrfLogError( OSRF_LOG_MARK,  "Session is NULL in app reqeust" ); return -1; }\
+	if(!d->method) { osrfLogError( OSRF_LOG_MARK,  "Method is NULL in app reqeust" ); return -1; }\
+	if(d->method->argc) {\
+		if(!d->params) { osrfLogError( OSRF_LOG_MARK,  "Params is NULL in app reqeust %s", d->method->name ); return -1; }\
+		if( d->params->type != JSON_ARRAY ) { \
+			osrfLogError( OSRF_LOG_MARK,  "'params' is not a JSON array for method %s", d->method->name);\
+			return -1; }\
+	}\
+	if( !d->method->name ) { osrfLogError( OSRF_LOG_MARK,  "Method name is NULL"); return -1; } 
+
+#ifdef OSRF_LOG_PARAMS 
+#define OSRF_METHOD_VERIFY_CONTEXT(d) \
+	_OSRF_METHOD_VERIFY_CONTEXT(d); \
+	char* __j = jsonObjectToJSON(d->params);\
+	if(__j) { \
+		osrfLogInfo( OSRF_LOG_MARK,  "CALL:	%s %s - %s", d->session->remote_service, d->method->name, __j);\
+		free(__j); \
+	} 
 #else
-#define OSRF_METHOD_VERIFY_CONTEXT(d) OSRF_METHOD_VERIFY_CONTEXT_(d);
+#define OSRF_METHOD_VERIFY_CONTEXT(d) _OSRF_METHOD_VERIFY_CONTEXT(d); 
 #endif
 
 
-- 
2.43.2