[
https://issues.apache.org/jira/browse/QPID-1153?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12617534#action_12617534
]
Steve Huston commented on QPID-1153:
------------------------------------
I attached a new patch that does:
- initialize min and max to their specified values (0, at this time)
- correctly set min on the first non-zero value seen
- rebased against current trunk version of schema.py
Please review and let me know if anything further is needed.
Thanks,
-Steve
> managementgen/schema.py initializes unsigned value with -1
> ----------------------------------------------------------
>
> Key: QPID-1153
> URL: https://issues.apache.org/jira/browse/QPID-1153
> Project: Qpid
> Issue Type: Bug
> Components: Code Generator
> Affects Versions: M3
> Environment: Red Hat Enterprise Linux 4, g++ 3.4.6
> Reporter: Steve Huston
> Assignee: Ted Ross
> Priority: Minor
> Attachments: schema.py.diff
>
>
> The cpp/managementgen/schema.py script generates
> gen/qpid/management/Queue.cpp that causes compile warnings which are promoted
> to errors by the build settings. The warnings are:
> gen/qpid/management/Queue.cpp: In constructor
> `qpid::management::Queue::Queue(qpid::management::Manageable*,
> qpid::management::Manageable*, std::string, uint8_t, uint8_t, uint8_t)':
> gen/qpid/management/Queue.cpp:80: warning: converting of negative value
> `-0x000000001' to `uint64_t'
> gen/qpid/management/Queue.cpp: In member function `virtual void
> qpid::management::Queue::writeStatistics(qpid::framing::Buffer&, bool)':
> gen/qpid/management/Queue.cpp:527: warning: converting of negative value
> `-0x000000001' to `uint64_t'
> The attached patch resolves the warnings by initializing using the value set
> in the spec. At this time, that turns out to be 0.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
