Re: PATCH: more sorting: multibyte case-insensitivity

[Peter Stephenson <pws@xxxxxxx>]

Bart Schaefer wrote:
> On Jan 22,  2:23pm, Peter Stephenson wrote:
> }
> } This extends the new sort code to handle lowering of case properly with
> } multibyte characters.
> 
> I'm beginning to worry that we're piling up some pretty severe performance
> penalties with this stuff.  Zsh was already about 1/3 the speed of bash on
> some benchmarks, as I recall.  Has anyone lately compared a recent CVS
> build to something a bit older?

"unsetopt MULTIBYTE" is supposed to get you pretty much back to where we
were before if you don't care about multibyte characters.  I'm not aware
of a general solution to the efficiency problem with proper multibyte
handling.

In this particular case, it should be more efficient than before, at
least with a long list to sort, since now the strings are transformed
once each instead of every time two strings are compared.

I haven't done any benchmarking.  At the moment I'm happy if it works.

-- 
Peter Stephenson <pws@xxxxxxx>                  Software Engineer
CSR PLC, Churchill House, Cambridge Business Park, Cowley Road
Cambridge, CB4 0WZ, UK                          Tel: +44 (0)1223 692070


To access the latest news from CSR copy this link into a web browser:  http://www.csr.com/email_sig.php

To get further information regarding CSR, please visit our Investor Relations page at http://ir.csr.com/csr/about/overview