Original Post
I added patch in patches tracker in sourceforge, but since there are only few patches I guess it is almost not used, so I decided to copy text here as well.
Here's link to patch https://sourceforge....752&atid=821100
In add_on/scriptstdstring ternary operator usage is wrong:
The '?:' operator has a lower priority than the '<<' operator, so this is equivalent to
and this is equivalent to
Proper variant is
I'm not sure if current version leads to any bugs, but it is definitely wrong. I attached patch, that add braces for ternary operators in scriptstdstring.
Here's link to patch https://sourceforge....752&atid=821100
In add_on/scriptstdstring ternary operator usage is wrong:
//bool b;
stream << b ? "true" : "false";The '?:' operator has a lower priority than the '<<' operator, so this is equivalent to
(stream << b) ? "true" : "false";and this is equivalent to
stream << b;Proper variant is
stream << (b ? "true" : "false");I'm not sure if current version leads to any bugs, but it is definitely wrong. I attached patch, that add braces for ternary operators in scriptstdstring.
