Identify and eliminate dead code (100+ lines) #6BRL-CAD
Status: ClosedTime to complete: 96 hrs Mentors: Popescu Andrei, IshwerdasTags: C, consolidation, code reduction, refactoring, cleanup

BRL-CAD is huge. With any large body of code, one inevitably ends up with a mix of good and bad coding practices. On the whole, BRL-CAD is actually better than most but we are constantly working on improving the code. This includes eliminating duplication.

References:

  • http://en.wikipedia.org/wiki/Don't_repeat_yourself
  • http://brlcad.org/wiki/Code_Cleanup
  • http://brlcad.org/wiki/Compiling
  • http://brlcad.org/wiki/SVN

This task involves reducing BRL-CAD's source code by 100 or more lines of code by refactoring and eliminating duplicate code. You can use whatever method you like to identify duplication, but beware that there are more than 1 million lines of code in BRL-CAD, so you're not likely going to find this duplication just by browsing.

We suggest using a code duplication detection tool like Simian. See our Code_Cleanup page for details.

Download our latest Subversion trunk sources and make sure you can compile cleanly first. Then you can run Simian or do whatever you need to find sources of code duplication. Make your edits, then make sure the code still compiles (run "make", "make test", "make regress" and "make benchmark" to test your changes). Finally, create and submit a patch file of your changes (see the references, svn will create the patch file for you).

Feel free to join the brlcad-devel mailing list or IRC channel to discuss your changes beforehand.

Uploaded Work
File name/URLFile sizeDate submitted
mged+rt.patch6.2 KBJanuary 17 2015 16:51 UTC
Comments
ihoegenon December 17 2014 23:11 UTCTask Claimed

I would like to work on this task.

ihoegenon December 17 2014 23:15 UTCClaim Removed

The claim on this task has been removed, someone else can claim it now.

Bruno P.on January 4 2015 03:36 UTCTask Claimed

I would like to work on this task.

Bruno P.on January 4 2015 04:21 UTCClaim Removed

The claim on this task has been removed, someone else can claim it now.

Bruno P.on January 4 2015 04:32 UTCTask Claimed

I would like to work on this task.

Gauravjeet Singh on January 4 2015 08:19 UTCTask Assigned

This task has been assigned to Bruno P.. You have 100 hours to complete this task, good luck!

Bruno P.on January 4 2015 14:19 UTCClaim Removed

The claim on this task has been removed, someone else can claim it now.

shardulcon January 16 2015 12:59 UTCTask Claimed

I would like to work on this task.

Popescu Andrei on January 16 2015 12:59 UTCTask Assigned

This task has been assigned to shardulc. You have 76 hours to complete this task, good luck!

shardulcon January 17 2015 16:51 UTCReady for review

The work on this task is ready to be reviewed.

Popescu Andrei on January 17 2015 18:23 UTCTask Closed

Congratulations, this task has been completed successfully.