TotalInsanity when using TH85 for small H2 model
When using the SET H2 SMALL MODEL TH85 command the code runs into TotalInsanity() in mole_drive.cpp. The if-statement that starts at line 158 has no clause for the TH85 case and therefore defaults to the TotalInsanity() else-clause.
The proposed fix is to start the if-statement with
if( hmi.chH2_small_model_type == ’T’ )
{
// do nothing
}
else if( hmi.chH2_small_model_type == 'H' )
{
// etc...
as the TH85 case was already handled before the start of the if-statement. It would also be prudent to add a test case exercising this branch in the test suite.
The attached test case shows the problem and fails on c17_branch as well as the trunk.
This problem was reported by Dries Van De Putte.