scan_number in string_utilities.cc should permit rational numbers, i.e. those with a decimal point.

[mholtrop]

Error is in the updated code. Corrected code, string_utilities.cc:54 is:

{
    // Scan the c_string str for numbers only, then return the value as a float.
    // The str is not allowed to have spaces or alphanumerics, only 0-9 and .
    int i=0;
//  while(char c=str[i++]) if((isalpha(c) || ispunct(c) || iscntrl(c) || isspace(c)) && !(c=='-' || c=='+' || c=='e' || c=='E') )
    while(char c=str[i++]) if(!(isdigit(c) || c=='-' || c=='+' || c=='e' || c=='E' || c=='.'))
    {
        cout << "WARNING: Unexpected character found in number string: " << str << endl;
        cout << "Exiting " << endl; exit(1);
    }
    return( stringToDouble(str));
}

[mholtrop]

Humbug. Seems:

{

    // Scan the c_string str for numbers only, then return the value as a float.
    // The str is not allowed to have spaces or alphanumerics, only 0-9 and .
    int i=0;
    while(char c=str[i++]) if(isalpha(c) && !(c=='-' || c=='+' || c=='e' || c=='E') )
    {
        cout << "WARNING: Unexpected Alphanumberic character found in number string:" << str << endl;
    }

    return( stringToDouble(str));

}

works as well. (Updated code a few hours ago.)

[zhaozhiwen]

this doesn't work when for "8,3e3" where "," is a mistake for "." while(char c=str[i++]) if(isalpha(c) && !(c=='-' || c=='+' || c=='e' || c=='E') )

this works better while(char c=str[i++]) if(!(isdigit(c) || c=='-' || c=='+' || c=='e' || c=='E' || c=='.'))

Mauri, can you fix this bug?

[mholtrop]

Ah yes, thanks Zhiwen, that would be data entries by our European collaborators (where . is exchanged with , in numbers sometimes).