BRL-CAD

Eliminate 50+ lines of code duplication

This task involves reducing source code by 50 or more lines of code by refactoring and eliminating duplicate code. You can use the sources to BRL-CAD, LibreCAD, or BRL-CAD's Online Geometry Viewer. You can use whatever method you like to identify duplication, but beware that you are not likely going to find duplication just by browsing.

With any large body of code, one inevitably ends up with a mix of good and bad coding practices but we are constantly working on improving. This includes eliminating code duplication. We suggest using a code duplication detection tool like Simian. See https://brlcad.org/wiki/Code_Cleanup#Duplication_Reduction for details.

Download the latest source code from repository and make sure you can compile:

Then you can run Simian or do whatever you need to find sources of code duplication. Make your edits, then make sure code still compiles and works.

SUBMIT your changes as a patch file or pull request.

Feel free to join IRC or a developer mailing list to discuss your changes beforehand.

Task tags

  • lua
  • javascript
  • tcl/tk
  • c/c++

Students who completed this task

Michal Hanus

Task type

  • code Code
  • done_all Quality Assurance
close

2016