On Sun, Jul 14, 2013 at 5:49 AM, Christopher Sean Morrison
<[email protected]> wrote:
> How did you go about finding and fixing that particular bit of duplication?  
> Looks like it would have been tricky to identify as duplication just by 
> reading the code.


I used Simian tool to find duplicacy. It showed only duplicate loops
in if-else structure.
But when I read the code, both loops were same in each case but
condition was reverse.
So I picked swapping technique there.

Tools are not enough to find duplicacy, code readability is also required.

--
Er. Avneet Kaur
Blog: www.avneetkhasla.wordpress.com

"Coming together is a beginning; keeping together is progress; working
together is success."

------------------------------------------------------------------------------
See everything from the browser to the database with AppDynamics
Get end-to-end visibility with application monitoring from AppDynamics
Isolate bottlenecks and diagnose root cause in seconds.
Start your free trial of AppDynamics Pro today!
http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk
_______________________________________________
BRL-CAD Developer mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/brlcad-devel

Reply via email to