Changeset 18239 for branches


Ignore:
Timestamp:
06/18/15 21:01:07 (9 years ago)
Author:
dneise
Message:
added notes about CheckUnderflow
File:
1 edited

Legend:

Unmodified
Added
Removed
  • branches/FACT++_scripts_refactoring/explicit_changelog

    r18234 r18239  
    166166
    167167# ----------------------------------------------------------------------
     168
     169Okay so I looked at CheckUnderflow.js. It was a pretty large mess, Which I broke up into
     17011 separate functions, none of them longer than one page.
     171But there was no possibility to reuse anything. So the code in total got larger.
     172Well, we will see if there is any possibility to reuse later, when refactoring is done.
     173
     174At least, I could imagine, that some of the function would in principle be testable,
     175while I have no idea, how to test anything, which is dim dependent, without the overhead
     176of writing a total mock up.
     177
     178Next up is: Statup.js, because this uses CheckUnderflow.
     179
     180Then I will look at all these handlers. Let's see.
     181
     182# ----------------------------------------------------------------------
Note: See TracChangeset for help on using the changeset viewer.