adapt to fixed log callbacks

- should be threadsafe now
- works for mip output now, too

src/gamsconfig.yaml.in

@@ -72,7 +72,7 @@ solverConfig:
 @COIN_HAS_HIGHS_TRUE@      libName: highs/libGamsHighs.la
 @COIN_HAS_HIGHS_TRUE@      auditCode: his
 @COIN_HAS_HIGHS_TRUE@      solverInterfaceType: 1
-@COIN_HAS_HIGHS_TRUE@      threadSafeIndic: False
+@COIN_HAS_HIGHS_TRUE@      threadSafeIndic: True
 @COIN_HAS_HIGHS_TRUE@    modelTypes:
 @COIN_HAS_HIGHS_TRUE@    - LP
 @COIN_HAS_HIGHS_TRUE@    - RMIP

src/highs/GamsHighs.cpp

@@ -25,17 +25,6 @@ typedef struct
    Highs*        highs;
 } gamshighs_t;
 
-static
-void gevprint(
-   HighsInt    level,
-   const char* msg,
-   void*       msgcb_data
-)
-{
-   gevHandle_t gev = (gevHandle_t) msgcb_data;
-   gevLogPChar(gev, msg);
-}
-
 static
 void gevlog(
    HighsLogType type,
@@ -114,14 +103,7 @@ int setupProblem(
    assert(gh->highs == NULL);
 
    gh->highs = new Highs();
-
-   // need to set log callbacks in options struct, since HiGHS::run() calls highsSetLogCallback(options)
-   // TODO we should not need to uncast const-ness, but there is no method in HiGHS class to set log callbacks
-   HighsOptions& options = const_cast<HighsOptions&>(gh->highs->getOptions());
-   options.printmsgcb = gevprint;
-   options.logmsgcb = gevlog;
-   options.msgcb_data = (void*) gh->gev;
-   highsSetLogCallback(options);
+   gh->highs->setLogCallback(gevlog, gh->gev);
 
    numCol = gmoN(gh->gmo);
    numRow = gmoM(gh->gmo);