| | 1 | = [wiki:ReviewHeuristicLab3.3.0Code HeuristicLab 3.3.0 Code Review] - !HeuristicLab.Optimizer-3.3 = |
| | 2 | |
| | 3 | == Reviewer: swinkler == |
| | 4 | |
| | 5 | * The years in the license header should be corrected |
| | 6 | * swagner: Done in r2790. |
| | 7 | * Several files do not contain a license statement |
| | 8 | * swagner: Added missing license information in r2790. |
| | 9 | * Using `System.Linq` might be unnecessary |
| | 10 | * swagner: Removed unused usings in r2790. |
| | 11 | * mkommend: I would not remove such essential although unused usings like `System.Linq`, because it is very likely that these will be added again. |
| | 12 | |
| | 13 | == Reviewer: mkommend == |
| | 14 | |
| | 15 | * '''!OperatorsMenuItem''': In the `OnToolStripItemSet` method a new `OperatorSideBar` is created and shown, which is a misuse of the `OnToolStripItemSet` method. If views should be created and shown at the appllication start up this functionality should be implemented in the `OnInitialized` method in the `MainForm` of the application. |
| | 16 | * swagner: Done in r2790 and r2791. |
| | 17 | |
| | 18 | * '''!NewItemDialog''': The items in the `ListView` should be aligned. |
| | 19 | * swagner: Done in r2790. |
