search

p-ranav / tabulate

Table Maker for Modern C++
MIT License
1.94k stars 138 forks source link

fix reverting locale in single_include #119

Open jedrezej opened 4 months ago

jedrezej commented 4 months ago

Looks like the author of pull-request #110 didn't change the single include file. This PR complements it.

I verified that this fixes an issue that I encountered in my tool which uses tabulate. The tool is supposed to use "C" locale, but after printing a 'tabulate' table, it gets changed to the environment locale (because format's default locale is "") and then subsequent writes to new files use the environment locale instead of "C".

@p-ranav - if you want I can include the changes from #110 here as well - let me know.

p-ranav commented 4 months ago

Yeah please include the changes from #110 and I can merge it all in one-go here. Thanks!

jedrezej commented 4 months ago

@p-ranav Done.