Changeset 18239 for branches/FACT++_scripts_refactoring/explicit_changelog

Timestamp:

06/18/15 21:01:07 (9 years ago)

Author:

dneise

Message:

added notes about CheckUnderflow

File:

• branches/FACT++_scripts_refactoring/explicit_changelog (modified) (1 diff)

branches/FACT++_scripts_refactoring/explicit_changelog

@@ -166 +166 @@
 
 # ----------------------------------------------------------------------
+
+Okay so I looked at CheckUnderflow.js. It was a pretty large mess, Which I broke up into
+11 separate functions, none of them longer than one page. 
+But there was no possibility to reuse anything. So the code in total got larger.
+Well, we will see if there is any possibility to reuse later, when refactoring is done.
+
+At least, I could imagine, that some of the function would in principle be testable,
+while I have no idea, how to test anything, which is dim dependent, without the overhead 
+of writing a total mock up.
+
+Next up is: Statup.js, because this uses CheckUnderflow.
+
+Then I will look at all these handlers. Let's see.
+
+# ----------------------------------------------------------------------