Identify and eliminate dead code (100+ lines) #4BRL-CAD
Status: ClosedTime to complete: 96 hrs Mentors: Jacob B, SeanTags: 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
_comb_12.2 KBDecember 17 2014 14:51 UTC
Comments
Jeffrey Voon December 2 2014 01:18 UTCTask Claimed

I would like to work on this task.

Mihai Neacsu on December 2 2014 01:19 UTCTask Assigned

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

Jeffrey Voon December 2 2014 02:10 UTCClaim Removed

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

Smrition December 2 2014 07:39 UTCTask Claimed

I would like to work on this task.

Sean on December 2 2014 08:10 UTCTask Assigned

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

Smrition December 2 2014 11:11 UTCClaim Removed

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

denver873on December 2 2014 17:11 UTCTask Claimed

I would like to work on this task.

denver873on December 2 2014 17:12 UTCClaim Removed

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

Amron December 2 2014 19:51 UTCTask Claimed

I would like to work on this task.

Deepak on December 2 2014 19:52 UTCTask Assigned

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

Amron December 2 2014 19:58 UTCClaim Removed

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

Martinon December 2 2014 20:14 UTCTask Claimed

I would like to work on this task.

Martinon December 2 2014 20:14 UTCClaim Removed

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

natashamon December 2 2014 21:59 UTCTask Claimed

I would like to work on this task.

Mandeep Kaur on December 2 2014 22:01 UTCTask Assigned

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

natashamon December 2 2014 22:08 UTCClaim Removed

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

Vladimir Kuznetsovon December 3 2014 08:08 UTCTask Claimed

I would like to work on this task.

Harmanpreet on December 3 2014 08:11 UTCTask Assigned

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

Melange on December 6 2014 12:11 UTCTask due soon

There are less than 24 hours left until the deadline, please submit your work soon.

Vladimir Kuznetsovon December 6 2014 19:03 UTCCommiting

OK, so i have made changes in two files, that remove ~110 lines of code. How do i upload these to your sourceforge?
I'm using windows 7.

Melange on December 7 2014 12:11 UTCTask Reopened

Melange has detected that the final deadline has passed and it has reopened the task.

xercsys003on December 11 2014 12:25 UTCTask Claimed

I would like to work on this task.

xercsys003on December 11 2014 12:26 UTCClaim Removed

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

Marc Tannouson December 17 2014 14:50 UTCTask Claimed

I would like to work on this task.

Daniel_R on December 17 2014 14:50 UTCTask Assigned

This task has been assigned to Marc Tannous. You have 96 hours to complete this task, good luck!

Marc Tannouson December 17 2014 14:51 UTCReady for review

The work on this task is ready to be reviewed.

Daniel_R on December 17 2014 14:55 UTCTask Closed

Congratulations, this task has been completed successfully.