ack, code review only./Thanks HansN
On 11/18/2015 12:35 PM, Gary Lee wrote:
> osaf/services/saf/amf/amfd/comp.cc | 7 +++++++
> 1 files changed, 7 insertions(+), 0 deletions(-)
>
>
> diff --git a/osaf/services/saf/amf/amfd/comp.cc
> b/osaf/services/saf/amf/amfd/comp.cc
> --- a/osaf/services/saf/amf/amfd/comp.cc
> +++ b/osaf/services/saf/amf/amfd/comp.cc
> @@ -1227,6 +1227,13 @@ static void comp_ccb_apply_modify_hdlr(s
> if (!strcmp(attribute->attrName, "saAmfCompType")) {
>
> SaNameT *dn = (SaNameT*) value;
> + const std::string
> oldType(Amf::to_string(&comp->saAmfCompType));
> + if (oldType.compare(Amf::to_string(dn)) == 0) {
> + // ignore 'change' if it's being set to the
> same value
> + TRACE("saAmfCompType '%s' unchanged for '%s'",
> comp->saAmfCompType.value,
> + opdata->objectName.value);
> + continue;
> + }
>
> TRACE("saAmfCompType modified from '%s' to '%s' for
> '%s'", comp->saAmfCompType.value, dn->value,
> opdata->objectName.value);
------------------------------------------------------------------------------
_______________________________________________
Opensaf-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/opensaf-devel