Thanks for your explanation!
Thank you for your implementation of the text module :)
> /* Get system default locale name and check it */
> locale_name = gl_locale_name(LC_CTYPE, "LC_CTYPE");
> - if((locale_name == NULL) || \
> - (strlen(locale_name)< 2))
> + if (locale_name == NULL)
> {
> PDF_DEBUG_BASE("Invalid locale info detected! '%s'",
> ((locale_name!=NULL) ? locale_name : "null"));
> return PDF_ETEXTENC;
> }
As you now just check if locale_name is NULL, there is no need to check
it again within the debug message. So the debug message could be just:
PDF_DEBUG_BASE("Invalid locale info detected! (null)");
All the other stuff seems to be ok for me.
Ok. The patch is installed in the trunk now.
Did you check the unit tests with this change?
Yes, I did.
Thanks.