---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15353/
---
(Updated Nov. 12, 2013, 9:42 a.m.)
Review request for qpid.
Changes
---
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15353/#review28738
---
Ship it!
Ship It!
- Andrew Stitcher
On Nov. 12, 2013, 9:42
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15353/
---
(Updated Nov. 11, 2013, 12:30 p.m.)
Review request for qpid.
Changes
---
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15353/#review28679
---
/trunk/qpid/cpp/src/qpid/broker/Broker.cpp
On Nov. 11, 2013, 7:31 p.m., Andrew Stitcher wrote:
/trunk/qpid/cpp/src/qpid/broker/Broker.cpp, line 220
https://reviews.apache.org/r/15353/diff/2/?file=381997#file381997line220
[Just my personal preference]
I think it would be clearer to make this:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15353/#review28699
---
Ship it!
- Andrew Stitcher
On Nov. 11, 2013, 12:30 p.m., Gordon
On Nov. 11, 2013, 7:31 p.m., Andrew Stitcher wrote:
/trunk/qpid/cpp/src/qpid/broker/Broker.cpp, line 220
https://reviews.apache.org/r/15353/diff/2/?file=381997#file381997line220
[Just my personal preference]
I think it would be clearer to make this:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15353/
---
Review request for qpid.
Bugs: QPID-5316
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15353/#review28552
---
/trunk/qpid/cpp/src/qpid/DataDir.cpp
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15353/#review28601
---
I can't really comment on the code. But as far as the functionality
10 matches
Mail list logo