Addressed following TODO: 
//TODO: clarify when we use reportNumber, reportUnit.getReportNum() - 1
and reportUnit.getReportNum(). [Freek]
[Pravin] reportUnit.getReportNum() - 1 represents report index. Report
index is used for array operations. 
Added reportIndex variable and getReportIndex() method in ReportUnit to
avoid repetitive calls to reportUnit.getReportNum() - 1.
Replaced reportUnit.getReportNum() - 1 with reportUnit.getReportIndex()
Replaced reportNum with reportIndex in ChartUnit.java
Adjusted ChartUnitTest.java for using reportIndex
Merge remote-tracking branch 'upstream/master'
Addressed following TODOs:
//TODO: perhaps it's easier to use the report number here as well, since
any unique id is ok? [Freek]  Using report index instead of report
number [Pravin] 
// TODO: this call to setSelected is needed after a refresh of the
report list? [Freek]  Made a comment next to this TODO [Pravin]
Merge pull request #18 from FreekDB/master

Fixed a bug in ViewerFrame.createOrUpdateSortPanel/createSortOptionPanel
Merge remote-tracking branch 'upstream/master'
ViewerFrame:
- Changed the updatePipelineStatus method to simply set the text of the statusField (instead of creating a new text field for each update.
- Created the getExtendedPipelineStatus method to have a single definition of the pipeline status description (called from the updatePipelineStatus and getControlFrame methods).
- Created the CHANGE_ROOT_DIRECTORY_COMMAND, SET_FILTER_COMMAND, SELECT_METRICS_COMMAND and ABOUT_COMMAND constants for the menu commands.
- Removed the layout for the statusPanel, since it appears to be unnecessary.
- Some minor textual changes.