smokephil/ardour
1
0
Fork 0
mirror of https://github.com/Ardour/ardour.git synced 2026-01-06 21:55:43 +01:00
Code Issues Projects Releases Wiki Activity

Use snprintf instead of std::stringstream when converting color values to strings

Browse source 
std::ostream/stringstream will use the current locale to determine the numeric
formatting. If the locale uses grouping then thousands separators will be
inserted in the output which produces an invalid color string in
UIConfiguration::reset_gtk_theme() and when converting colors to strings in
UIConfiguration::store_color_theme()

This has not been a problem so far because it appears that LocaleGuard does not
reset the LC_NUMERIC value for the global C++ locale. So if a LocaleGuard is
created at any time before these functions are called(even if it goes out of
scope) the numeric formatting used by std::streams will use the "C" locale
formatting facets.
This commit is contained in:
Tim Mayberry 2016-09-06 14:27:10 +10:00
parent 0656d5ab1a
commit e7a23eaaa8
2 changed files with 35 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

3
gtk2_ardour/ui_config.h
View file

@ -83,6 +83,9 @@ private:
ArdourCanvas::HSV color_hsv (const std::string&) const;
ArdourCanvas::SVAModifier modifier (const std::string&) const;
static std::string color_to_hex_string (ArdourCanvas::Color c);
static std::string color_to_hex_string_no_alpha (ArdourCanvas::Color c);
sigc::signal<void> ColorsChanged;
void reset_dpi ();