Hi, immOmIsLoader has been recently changed (#8434). I must be checking of earlier OpenSAF release.
Ignore immOmIsLoader comment. Thanks, Zoran From: Hung Nguyen [mailto:[email protected]] Sent: den 20 december 2016 08:55 To: Zoran Milinkovic <[email protected]>; [email protected] Cc: [email protected] Subject: Re: [PATCH 1 of 3] imm: Remove use of SaBoolT from library [#2225] Hi Zoran, In my repository, that 'immOmIsLoader' variable already has the type of bool. (default branch) Can you check if your repository is up to date? Thanks, Hung Nguyen - DEK Technologies -------------------------------------------------------------------------------- From: Zoran Milinkovic [email protected]<mailto:[email protected]> Sent: Monday, December 19, 2016 6:55PM To: Hung Nguyen, Neelakanta Reddy [email protected]<mailto:[email protected]>, [email protected]<mailto:[email protected]> Cc: Opensaf-devel [email protected]<mailto:[email protected]> Subject: RE: [PATCH 1 of 3] imm: Remove use of SaBoolT from library [#2225] Hi Hung, In imma_om_api.c in line 52: 'static SaBoolT immOmIsLoader = SA_FALSE;' ... can be replaced with bool type and false. Thanks, Zoran -----Original Message----- From: Hung Nguyen [mailto:[email protected]] Sent: den 14 december 2016 09:46 To: Zoran Milinkovic <[email protected]><mailto:[email protected]>; [email protected]<mailto:[email protected]> Cc: [email protected]<mailto:[email protected]> Subject: [PATCH 1 of 3] imm: Remove use of SaBoolT from library [#2225] osaf/libs/agents/saf/imma/imma_oi_api.cc | 6 ++-- osaf/libs/agents/saf/imma/imma_om_api.cc | 48 ++++++++++++++++---------------- 2 files changed, 27 insertions(+), 27 deletions(-) imm: Remove use of SaBoolT from library. diff --git a/osaf/libs/agents/saf/imma/imma_oi_api.cc b/osaf/libs/agents/saf/imma/imma_oi_api.cc --- a/osaf/libs/agents/saf/imma/imma_oi_api.cc +++ b/osaf/libs/agents/saf/imma/imma_oi_api.cc @@ -3263,7 +3263,7 @@ static SaAisErrorT rt_object_delete_comm cb_lock must NOT be locked on entry. */ -static SaBoolT imma_implementer_set(IMMA_CB *cb, SaImmOiHandleT immOiHandle) +static bool imma_implementer_set(IMMA_CB *cb, SaImmOiHandleT immOiHandle) { SaAisErrorT err = SA_AIS_OK; unsigned int sleep_delay_ms = 200; @@ -3328,12 +3328,12 @@ static SaBoolT imma_implementer_set(IMMA success: osafassert(!locked); TRACE_LEAVE(); - return SA_TRUE; + return true; fail: if (locked) {m_NCS_UNLOCK(&cb->cb_lock, NCS_LOCK_WRITE);} TRACE_LEAVE(); - return SA_FALSE; + return false; } int imma_oi_resurrect(IMMA_CB *cb, IMMA_CLIENT_NODE *cl_node, bool *locked, SaAisErrorT * err_cli_res) diff --git a/osaf/libs/agents/saf/imma/imma_om_api.cc b/osaf/libs/agents/saf/imma/imma_om_api.cc --- a/osaf/libs/agents/saf/imma/imma_om_api.cc +++ b/osaf/libs/agents/saf/imma/imma_om_api.cc @@ -2139,7 +2139,7 @@ static SaAisErrorT ccb_object_modify_com SaAisErrorT saImmOmCcbObjectModify_2(SaImmCcbHandleT ccbHandle, const SaNameT *objectName, const SaImmAttrModificationT_2 **attrMods) { - SaBoolT freeMemory = SA_FALSE; + bool freeMemory = false; SaStringT objectNameStr = NULL; SaAisErrorT rc; @@ -2154,7 +2154,7 @@ SaAisErrorT saImmOmCcbObjectModify_2(SaI objectNameStr = (SaStringT)malloc(len + 1); memcpy(objectNameStr, osaf_extended_name_borrow(objectName), len); objectNameStr[len] = 0; - freeMemory = SA_TRUE; + freeMemory = true; } else { objectNameStr = (SaStringT)osaf_extended_name_borrow(objectName); } @@ -2162,7 +2162,7 @@ SaAisErrorT saImmOmCcbObjectModify_2(SaI rc = ccb_object_modify_common(ccbHandle, objectNameStr, attrMods, false); - if(freeMemory == SA_TRUE) { + if(freeMemory) { free(objectNameStr); } @@ -2633,7 +2633,7 @@ static SaAisErrorT ccb_object_delete_com SaAisErrorT saImmOmCcbObjectDelete(SaImmCcbHandleT ccbHandle, const SaNameT *objectName) { - SaBoolT freeMemory = SA_FALSE; + bool freeMemory = false; SaStringT objectNameStr = NULL; SaAisErrorT rc; @@ -2648,7 +2648,7 @@ SaAisErrorT saImmOmCcbObjectDelete(SaImm objectNameStr = (SaStringT)malloc(len + 1); memcpy(objectNameStr, osaf_extended_name_borrow(objectName), len); objectNameStr[len] = 0; - freeMemory = SA_TRUE; + freeMemory = true; } else { objectNameStr = (SaStringT)osaf_extended_name_borrow(objectName); } @@ -2656,7 +2656,7 @@ SaAisErrorT saImmOmCcbObjectDelete(SaImm rc = ccb_object_delete_common(ccbHandle, objectNameStr, false); - if(freeMemory == SA_TRUE) { + if(freeMemory) { free(objectNameStr); } @@ -3563,7 +3563,7 @@ SaAisErrorT saImmOmAdminOperationInvoke_ SaAisErrorT *operationReturnValue, SaTimeT timeout) { - SaBoolT freeMemory = SA_FALSE; + bool freeMemory = false; SaStringT objectNameStr = NULL; SaAisErrorT rc; @@ -3578,7 +3578,7 @@ SaAisErrorT saImmOmAdminOperationInvoke_ objectNameStr = (SaStringT)malloc(len + 1); memcpy(objectNameStr, osaf_extended_name_borrow(objectName), len); objectNameStr[len] = 0; - freeMemory = SA_TRUE; + freeMemory = true; } else { objectNameStr = (SaStringT)osaf_extended_name_borrow(objectName); } @@ -3587,7 +3587,7 @@ SaAisErrorT saImmOmAdminOperationInvoke_ rc = admin_op_invoke_common(ownerHandle, objectNameStr, continuationId, operationId, params, operationReturnValue, timeout, NULL, false, false); - if(freeMemory == SA_TRUE) { + if(freeMemory) { free(objectNameStr); } @@ -3604,7 +3604,7 @@ SaAisErrorT saImmOmAdminOperationInvoke_ SaTimeT timeout, SaImmAdminOperationParamsT_2 ***returnParams) { - SaBoolT freeMemory = SA_FALSE; + bool freeMemory = false; SaStringT objectNameStr = NULL; SaAisErrorT rc; @@ -3619,7 +3619,7 @@ SaAisErrorT saImmOmAdminOperationInvoke_ objectNameStr = (SaStringT)malloc(len + 1); memcpy(objectNameStr, osaf_extended_name_borrow(objectName), len); objectNameStr[len] = 0; - freeMemory = SA_TRUE; + freeMemory = true; } else { objectNameStr = (SaStringT)osaf_extended_name_borrow(objectName); } @@ -3628,7 +3628,7 @@ SaAisErrorT saImmOmAdminOperationInvoke_ rc = admin_op_invoke_common(ownerHandle, objectNameStr, continuationId, operationId, params, operationReturnValue, timeout, returnParams, true, false); - if(freeMemory == SA_TRUE) { + if(freeMemory) { free(objectNameStr); } @@ -4174,7 +4174,7 @@ SaAisErrorT saImmOmAdminOperationInvokeA SaImmAdminOperationIdT operationId, const SaImmAdminOperationParamsT_2 **params) { - SaBoolT freeMemory = SA_FALSE; + bool freeMemory = false; SaStringT objectNameStr = NULL; SaAisErrorT rc; @@ -4189,7 +4189,7 @@ SaAisErrorT saImmOmAdminOperationInvokeA objectNameStr = (SaStringT)malloc(len + 1); memcpy(objectNameStr, osaf_extended_name_borrow(objectName), len); objectNameStr[len] = 0; - freeMemory = SA_TRUE; + freeMemory = true; } else { objectNameStr = (SaStringT)osaf_extended_name_borrow(objectName); } @@ -4198,7 +4198,7 @@ SaAisErrorT saImmOmAdminOperationInvokeA rc = admin_op_invoke_async_common(ownerHandle, userInvocation, objectNameStr, continuationId, operationId, params, false); - if(freeMemory == SA_TRUE) { + if(freeMemory) { free(objectNameStr); } @@ -5704,7 +5704,7 @@ SaAisErrorT saImmOmAccessorGet_2(SaImmAc const SaNameT *objectName, const SaImmAttrNameT *attributeNames, SaImmAttrValuesT_2 ***attributes) { - SaBoolT freeMemory = SA_FALSE; + bool freeMemory = false; SaStringT objectNameStr = NULL; SaAisErrorT rc; @@ -5719,7 +5719,7 @@ SaAisErrorT saImmOmAccessorGet_2(SaImmAc objectNameStr = (SaStringT)malloc(len + 1); memcpy(objectNameStr, osaf_extended_name_borrow(objectName), len); objectNameStr[len] = 0; - freeMemory = SA_TRUE; + freeMemory = true; } else { objectNameStr = (SaStringT)osaf_extended_name_borrow(objectName); } @@ -5727,7 +5727,7 @@ SaAisErrorT saImmOmAccessorGet_2(SaImmAc rc = accessor_get_common(accessorHandle, objectNameStr, attributeNames, attributes, false, 0); - if(freeMemory == SA_TRUE) { + if(freeMemory) { free(objectNameStr); } @@ -6956,7 +6956,7 @@ SaAisErrorT saImmOmSearchInitialize_2(Sa const SaImmSearchParametersT_2 *searchParam, const SaImmAttrNameT *attributeNames, SaImmSearchHandleT *searchHandle) { - SaBoolT freeMemory = SA_FALSE; + bool freeMemory = false; SaStringT rootNameStr = NULL; SaAisErrorT rc; @@ -6971,7 +6971,7 @@ SaAisErrorT saImmOmSearchInitialize_2(Sa rootNameStr = (SaStringT)malloc(len + 1); memcpy(rootNameStr, osaf_extended_name_borrow(rootName), len); rootNameStr[len] = 0; - freeMemory = SA_TRUE; + freeMemory = true; } else { rootNameStr = (SaStringT)osaf_extended_name_borrow(rootName); } @@ -6980,7 +6980,7 @@ SaAisErrorT saImmOmSearchInitialize_2(Sa rc = search_init_common(immHandle, rootNameStr, scope, searchOptions, searchParam, attributeNames, searchHandle, false); - if(freeMemory == SA_TRUE) { + if(freeMemory) { free(rootNameStr); } @@ -9079,7 +9079,7 @@ SaAisErrorT saImmOmCcbFinalize(SaImmCcbH cb must NOT be locked on entry. */ -static SaBoolT imma_re_initialize_admin_owners(IMMA_CB *cb, SaImmHandleT immHandle) +static bool imma_re_initialize_admin_owners(IMMA_CB *cb, SaImmHandleT immHandle) { SaUint32T proc_rc = NCSCC_RC_SUCCESS; SaAisErrorT err = SA_AIS_OK; @@ -9232,13 +9232,13 @@ static SaBoolT imma_re_initialize_admin_ if (locked) {m_NCS_UNLOCK(&cb->cb_lock, NCS_LOCK_WRITE);} if (out_evt) {free(out_evt);} TRACE_LEAVE(); - return SA_TRUE; + return true; fail: if (locked) {m_NCS_UNLOCK(&cb->cb_lock, NCS_LOCK_WRITE);} if (out_evt) {free(out_evt);} TRACE_LEAVE(); - return SA_FALSE; + return false; } int imma_om_resurrect(IMMA_CB *cb, IMMA_CLIENT_NODE *cl_node, bool *locked) ------------------------------------------------------------------------------ Developer Access Program for Intel Xeon Phi Processors Access to Intel Xeon Phi processor-based developer platforms. With one year of Intel Parallel Studio XE. Training and support from Colfax. Order your platform today.http://sdm.link/intel _______________________________________________ Opensaf-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/opensaf-devel
