Em Qua, 2013-10-16 às 20:28 +0200, the Adib escreveu:
I think it should be :
bool viewBox_set = true;
Are you talking about C++11's non-static data member initialization[1]? The SPRoot's constructor disagrees. It is explicitly setting the viewBox_set value to FALSE (yes, in capital, probably a macro). And I think currently Inkscape is compiled using C++98/03.
Indeed, the SPRoot's code is a bit weird.
About the change to more idiomatic code, I don't know what is the Inkscape rules regarding bitfields. I hope some older developer will answer this one.