Alexander Barkov wrote:
> That's true , <tab> in many collations is the smallest character
> (i.e. minSortChar=<tab>), and space is greater than <tab>.
> So does not seem correct to remove both spaces and tabs, and
> then execute strnncoll() for the results in the patch for
You're absolutely right.
We should not remove tabs, but only spaces. I don't know what
the code does currently, but the original design (not necessarily
its implementation) removed only spaces. (And, of course, trailing
zeros for numerics, but that's another topic.)