search

modelica / ModelicaStandardLibrary

Free (standard conforming) library to model mechanical (1D/3D), electrical (analog, digital, machines), magnetic, thermal, fluid, control systems and hierarchical state machines. Also numerical functions and functions for strings, files and streams are included.
https://doc.modelica.org
BSD 3-Clause "New" or "Revised" License
471 stars 168 forks source link

skipWhiteSpace under-documented #3820

Open DagBruck opened 3 years ago

DagBruck commented 3 years ago

The function Modelica.Utilities.Strings.Advanced.skipWhiteSpace appears to be under-documented. The cases I have found are:

  1. What does it return when the given string contains only whitespace or is an empty string?
  2. What does it return if the start index is greater than the string length.

Here are three test cases.

Modelica.Utilities.Strings.Advanced.skipWhiteSpace("")
Modelica.Utilities.Strings.Advanced.skipWhiteSpace("  ")
Modelica.Utilities.Strings.Advanced.skipWhiteSpace(" a ", 700)
DagBruck commented 3 years ago 
Modelica.Utilities.Strings.Advanced.skipWhiteSpace("")
Modelica.Utilities.Strings.Advanced.skipWhiteSpace("  ")
Modelica.Utilities.Strings.Advanced.skipWhiteSpace(" a ", 700)

Might add that it returns 1, 3 and 700 for these cases. The last one in particular is questionable.

sjoelund commented 3 years ago 
Modelica.Utilities.Strings.Advanced.skipWhiteSpace("")
Modelica.Utilities.Strings.Advanced.skipWhiteSpace("  ")
Modelica.Utilities.Strings.Advanced.skipWhiteSpace(" a ", 700)

Might add that it returns 1, 4 and 700 for these cases. The last one in particular is questionable.

Is that a Windows thing? It returns 1, 3 and 700 for me on Linux...

DagBruck commented 3 years ago

Might add that it returns 1, 4 and 700 for these cases. The last one in particular is questionable.

Is that a Windows thing? It returns 1, 3 and 700 for me on Linux...

No a typo on my part :-)