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.

[1] http://en.wikipedia.org/wiki/C++11#Object_construction_improvement

Vinícius dos Santos Oliveira