swami / libinstpatch

Instrument file software library.
Other
20 stars 6 forks source link

Integrate setlocale() in libinstpatch dll: #26

Closed jjceresa closed 5 years ago

jjceresa commented 5 years ago

This fix a side effect introduced by commit b45b2db for which Windows libinstpatch dll is linked with static version of runtime libraries (option Multithread (/MT)) instead of DLL multithread option (/MD). In this case, even if the application call setlocale() function before calling any ipatch_xml_xxxx_decode_xxxx_func(), these decode functions will ignore the effect of setlocale() leading in float number not properly decoded. This commit fix this issue by integrating call to setlocale() inside ipatch_init().

derselbst commented 5 years ago

Ok, good to know. Thanks.