Changes between Version 4 and Version 5 of ReviewHeuristicLab3.3.0CodeCore

Timestamp:

04/09/10 14:11:33 (15 years ago)

Author:

mkommend

Comment:

--

ReviewHeuristicLab3.3.0CodeCore

@@ -20 +20 @@
  * '''Engine''': A better name for the method {{{public void Initialize()}}} (line 166) and the according event handler should be chosen. Initialized suggests that the method should only be called once after the creation of an instance. A solution would be to make the {{{Initialize}}} method private and a public {{{Reset}}} method which calls the {{{Initialize}}} method.
    * swagner: Renamed `Initialize` to `Prepare` and `Initialized` to `Prepared` in r2790.
+ * '''Engine''': Currently the elapsed execution time is calculated by subtracting `DateTime.Now` - lastUpdated. IMHO using the `StopWatch` class would be more appropriate for this task.
 
  * '''!ExecutionContext''': The {{{private IOperator op;}}} should be renamed to operator to match the corresponding property.