Ack, code review only/Thanks HansN -----Original Message----- From: Gary Lee [mailto:gary....@dektech.com.au] Sent: den 15 september 2015 09:29 To: Hans Nordebäck; nagendr...@oracle.com; Gary Lee Cc: opensaf-devel@lists.sourceforge.net Subject: [PATCH 1 of 1] amfd: remove unused variables [#1142]
osaf/services/saf/amf/amfd/sg_nored_fsm.cc | 2 -- osaf/services/saf/amf/amfd/sg_nwayact_fsm.cc | 2 -- 2 files changed, 0 insertions(+), 4 deletions(-) This fixes unused variable warnings if both 'convert list_of_su to std::set V2' and 'convert AVD_SG_OPER to std::list V2' refactoring patches are applied. diff --git a/osaf/services/saf/amf/amfd/sg_nored_fsm.cc b/osaf/services/saf/amf/amfd/sg_nored_fsm.cc --- a/osaf/services/saf/amf/amfd/sg_nored_fsm.cc +++ b/osaf/services/saf/amf/amfd/sg_nored_fsm.cc @@ -1067,8 +1067,6 @@ uint32_t SG_NORED::si_admin_down(AVD_CL_ } uint32_t SG_NORED::sg_admin_down(AVD_CL_CB *cb, AVD_SG *sg) { - AVD_SU *i_su; - TRACE_ENTER2("%u", sg->sg_fsm_state); if ((cb->init_state != AVD_APP_STATE) && (sg->sg_ncs_spec == false)) { diff --git a/osaf/services/saf/amf/amfd/sg_nwayact_fsm.cc b/osaf/services/saf/amf/amfd/sg_nwayact_fsm.cc --- a/osaf/services/saf/amf/amfd/sg_nwayact_fsm.cc +++ b/osaf/services/saf/amf/amfd/sg_nwayact_fsm.cc @@ -1659,8 +1659,6 @@ uint32_t SG_NACV::si_admin_down(AVD_CL_C } uint32_t SG_NACV::sg_admin_down(AVD_CL_CB *cb, AVD_SG *sg) { - AVD_SU *i_su; - TRACE_ENTER2("%u", sg->sg_fsm_state); if ((cb->init_state != AVD_APP_STATE) && (sg->sg_ncs_spec == false)) { ------------------------------------------------------------------------------ _______________________________________________ Opensaf-devel mailing list Opensaf-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/opensaf-devel