I had a look on the set property of the SPIPaint structure, and I noticed it is only used as a bool, while this property was defined as an unsigned integer. This is not a good programming practice imho.
This results in comparing this integer with an a boolean in amongst others desktop-style.cpp line 505. Is this desirable?
> Revision 12297 changed several bitfields in SPStyle to bool and unsigned int.
As far as I can see only one field is changed from unsigned integer to bool, so the SPIPaint memory size should decrease. Unless, I overlook something in the code. In that cas, I'd be happy to learn something new. ;-)
Regards, K.