Affects Version/s: None
Fix Version/s: 1.1
Patch Info:Patch Available
The AMQCPP code looks great in general, but many files have some stray whitespace. Also, there are quite a few headers that contain member function definitions. Some are simple, but there's also some big ones in there. I usually follow the rule that anything that can go into the .cpp should.
Attached is a patch to clean up the files in activemq::concurrent. If Timothy/Nathan agrees that doing this is useful, I'll submit patches for the rest.
|Field||Original Value||New Value|
|Attachment||StompFrame-signmismatch.diff [ 15260 ]|
|Assignee||Nathan Mittler [ nmittler ]||Timothy Bish [ tabish121 ]|
|Resolution||Fixed [ 1 ]|
|Status||Open [ 1 ]||Resolved [ 5 ]|
|Project Import||Fri Nov 26 23:25:50 EST 2010 [ 1290831950293 ]|
|Transition||Time In Source Status||Execution Times||Last Executer||Last Execution Date|
|3d 6h 29m||1||Timothy Bish||28/Dec/06 14:22|