Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

A lot of the NetarchiveSuite source code is written with very compact naming of variables and methods, which doesn't include much information about the purpose of the variable or methods. The result is that the code can become very  very difficult to read. This has sometimes has been mitigated by adding inline comments. Writing readable code is in my opinion is fare superior to incomprehensible code riddled with comments.

Concrete example can be found in the HarvestSchedulerTester#testSubmitNewJobsMakesDuplicateReductionInfo method:

...