Re: [xslt] Rework locale feature tests

[Nick Wellnhofer <wellnhofer aevum de>]

On 09/01/2018 12:47, Vojtech Fried wrote:
> > I think there never was a configure option for locale support. It certainly
> > isn't a feature that was removed.
> I would have to check for unix, but for windows, there was a support for turning locales off (via configure.js). And it 
> was removed in the commit "Rework locale feature tests".

For Windows, that's possible. I was thinking about UNIX.

> > A custom sort function overrides the internal sort function. It shouldn't
> > matter whether libxslt is compiled with locale support or not.
> My sorting function (slightly modified version of libxslt/examples/ xsltICUSort.c) internally calls 
> xsltComputeSortResult, which uses a locale if set. Maybe my sorting function could deal with it (e.g. it 
> could have its own version of xsltComputeSortResult), but the easiest way was to turn locale support off 
> completely. I do not need it, since I do the sort myself.

I see. I'd suggest to use your own version of xsltComputeSortResult from 
libxslt/xsltutils.c and remove the following lines:

            if (comp->locale != (xsltLocale)0) {
                xmlChar *str = res->stringval;
                res->stringval = (xmlChar *) xsltStrxfrm(comp->locale, str);
                xmlFree(str);
            }

Nick