This is great initiative Babak, thanks. I have also started some code cleanup activity on my own for the 528 critical violations reported on Camel Sonar, but didn't go too far for now.
Bilgin On 14 December 2011 19:36, bvahdat <babak.vah...@swissonline.ch> wrote: > Hi Christian, > >> May you can start with this after we released Camel 2.9.0 and split the >> work into multiple, not so huge patches, e.g. one issue per point on your >> list. > > Yeah, exactly. Instead of attaching a single huge diff on the ticket I > intend to do it gradually (for example one patch for each single issue I've > mentioned already), so that the dialog in that ticket flows as something > like: > > My Comment: attached CAMEL-xyz-1.patch, so please verify and > commit > Commiter's Comment: applied the patch CAMEL-xyz-1.patch into the trunk > > I update my workspace with that (and maybe other) revision(s) (as well). > > My Comment: attached CAMEL-xyz-2.patch, so please verify and > commit > Commiter's Comment: applied the patch CAMEL-xyz-2.patch into the trunk > > And so on and so on... until we hit all of them. > > Babak > > > -- > View this message in context: > http://camel.465427.n5.nabble.com/DISCUSS-Trunk-Code-Cleanup-tp5071594p5075451.html > Sent from the Camel Development mailing list archive at Nabble.com.
