Qpid
  1. Qpid
  2. QPID-4449

Fix for QPID-4286 changed the standalone qmf-agent API

    Details

      Description

      The changes to qpid/trunk/qpid/cpp/include/qpid/management/Manageable.h cause applications that use the stand-alone qmf agent API to need to adapt.

      This file needs to be changed to provide backward compatibility for users of the API.

        Activity

        Hide
        Justin Ross added a comment -

        http://svn.apache.org/viewvc?view=revision&revision=1411761

        Ken, please review for 0.20 inclusion.

        Show
        Justin Ross added a comment - http://svn.apache.org/viewvc?view=revision&revision=1411761 Ken, please review for 0.20 inclusion.
        Hide
        Ken Giusti added a comment -

        Justin/Ted - is the intent to deprecate the old non-shared-ptr API in favor of the new shared-ptr API?

        The changes look fine, btw - just curious as to the intent of the new method.

        -K

        Show
        Ken Giusti added a comment - Justin/Ted - is the intent to deprecate the old non-shared-ptr API in favor of the new shared-ptr API? The changes look fine, btw - just curious as to the intent of the new method. -K
        Hide
        Ted Ross added a comment -

        Ken,

        The new shared-ptr interface is needed for the agent-in-the-broker. The original change forced all users to move to the shared-ptr API, including users of the stand-alone QMF Agent. QPID-4449 reverts the stand-alone agent API to be unchanged while allowing the in-broker agent to use the shared-ptr API for easier memory management of objects.

        -Ted

        Show
        Ted Ross added a comment - Ken, The new shared-ptr interface is needed for the agent-in-the-broker. The original change forced all users to move to the shared-ptr API, including users of the stand-alone QMF Agent. QPID-4449 reverts the stand-alone agent API to be unchanged while allowing the in-broker agent to use the shared-ptr API for easier memory management of objects. -Ted
        Hide
        Justin Ross added a comment -

        Reviewed by Ken. Approved for 0.20.

        Show
        Justin Ross added a comment - Reviewed by Ken. Approved for 0.20.

          People

          • Assignee:
            Ken Giusti
            Reporter:
            Ted Ross
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development