Fix potential UB in LTV controllers

[virtuald]

• If the constructor arguments are incorrect, the table will have zero entries, which will access invalid iterators in interpolating_map::operator[]

Only bad usages of #5139 I could see

[virtuald]

/format

[calcmogul]

This change should be made in Java as well.

[calcmogul]

I'm guessing what happened is you gave a random 2x2 linear system that had a zero or negative steady-state velocity (maxV <= 0). If that can be confirmed, we can mention that in the error message to make it more informative and throw the exception before the for loop.

[virtuald]

I didn't run into this error, I just went looking for incorrect uses of std::*_bound

[calcmogul]

Well, I'd prefer to tell the user what actually caused the error. The interpolating map being empty is just a symptom of a violated precondition earlier in the constructor.