[Scummvm-devel] Make class String case insensitive by default?

[Marcus Comstedt]

"Johannes Schickel" <lordhoto at gmail.com> writes:

> I thought of adding one bool which is used for the mode to use, this
> would of course need branching for every comparison function, but it
> shouldn't be that slow nor a big waste of memory.

4 bytes per string adds up if there are many strings.  The subclass
proposal is better as it relies on the vtable field which is already
there.  Also it is less error prone as you don't add extra state to
the strings which needs to be maintained.

Even so, I agree with Oystein that sensitivity should be a property of
the comparison, not the string.


  // Marcus