Re: [devel] [PATCH 1 of 1] * remove saAmfStgValidSuTypes from 'struct avd_amf_sg_type_tag' as it appears to

2014-02-03 Thread Hans Feldt
Ack with minor comments: * commit message too long, amfd: fix valgrind reported errors [#715] is enough for me * check pointer with NULL, see inline Thanks, Hans On 01/13/2014 11:53 AM, Gary Lee wrote: osaf/services/saf/amf/amfd/csi.cc | 25 -

[devel] [PATCH 1 of 1] * remove saAmfStgValidSuTypes from 'struct avd_amf_sg_type_tag' as it appears to

2014-01-13 Thread Gary Lee
osaf/services/saf/amf/amfd/csi.cc | 25 - osaf/services/saf/amf/amfd/include/sg.h | 1 - osaf/services/saf/amf/amfd/sgtype.cc| 3 ++- osaf/services/saf/amf/amfd/si.cc| 27 +-- osaf/services/saf/amf/amfd/svctype.cc | 20

Re: [devel] [PATCH 1 of 1] * remove saAmfStgValidSuTypes from 'struct avd_amf_sg_type_tag' as it appears to

2014-01-13 Thread Gary Lee
Hmm, something went wrong when the email was done. The commit message should read: * remove saAmfStgValidSuTypes from 'struct avd_amf_sg_type_tag' as it appears to be a typo of saAmfSGtValidSuTypes * ensure saAmfCSIDependencies is deleted in csi.cc * ensure