Hello Andrew,

I understand that you do not want to create env before checking if TLS
is configured. But, please, do not call abort() in glp_config().

Otherwise your abort statement will stop the complete application.
Imagine that being the webserver for a site on which a webapp is
deployed which calls the GLPK library.

Either simply return NULL. Or define a value with signifies an error:

glpk.h:
#define GLP_INVALID ((const void *) -1)

env/env.c:
const char *glp_config(const char *option)
{
...
   return GLP_INVALID;
...
}

Best regards

Heinrich Schuchardt

On 01/14/2017 03:27 AM, Andrew Makhorin wrote:
>> a program that uses a GLPK library which is not compiled with thread
>> local storage may fail fatally when run with multiple threads.
>>
>> I hence suggest to add a function that allows to determine if GLPK was
>> compiled to be reentrant. We could use the same function to return other
>> features too.
> 
> Done. Please see an updated version of glpk here:
> http://sourceforge.net/projects/noumenon/files/tmp/
> (Note that this is *not* an official release.)
> 
> I added API routine glp_config (see it in src/env/env.c) which provides
> necessary functionality.
> 
> 
> Andrew Makhorin
> 
> 


_______________________________________________
Help-glpk mailing list
Help-glpk@gnu.org
https://lists.gnu.org/mailman/listinfo/help-glpk

Reply via email to