[ 
https://issues.apache.org/jira/browse/QPID-3335?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13058556#comment-13058556
 ] 

[email protected] commented on QPID-3335:
-----------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/988/
-----------------------------------------------------------

Review request for Gordon Sim, Kim van der Riet and Ted Ross.


Summary
-------

Removes some old code that duplicates type conversions that are available in 
the common library.


This addresses bug qpid-3335.
    https://issues.apache.org/jira/browse/qpid-3335


Diffs
-----

  /trunk/qpid/cpp/src/qpid/management/ManagementAgent.h 1141932 
  /trunk/qpid/cpp/src/qpid/management/ManagementAgent.cpp 1141932 

Diff: https://reviews.apache.org/r/988/diff


Testing
-------

unit.


Thanks,

Kenneth



> Remove duplicate and buggy type conversion code from Management Agent.
> ----------------------------------------------------------------------
>
>                 Key: QPID-3335
>                 URL: https://issues.apache.org/jira/browse/QPID-3335
>             Project: Qpid
>          Issue Type: Bug
>          Components: C++ Broker
>    Affects Versions: 0.11
>            Reporter: Ken Giusti
>            Assignee: Ken Giusti
>             Fix For: 0.12
>
>
> The broker's management agent exports a FieldTable to Variant Map translation 
> API.  This duplicates the translation code from the common library 
> (Codecs.h), and the version in the management agent is stale and buggy.   
> Remove the duplication and have the management agent use the common code. 

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:[email protected]

Reply via email to