Re: Rework locale feature tests

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