typo in rgba.c rgba_clear

[daym]

Fixed by

@@ -226,7 +231,7 @@ void rgba_clear(rgba_surface_t *rgba)
                return;

        rgba_fill(rgba, &rgba->dirty, 0x00000000);
-       rgba->flags &= ~(RGBA_FLAG_DIRTY || RGBA_FLAG_NEEDS_CLEAR);
+       rgba->flags &= ~(RGBA_FLAG_DIRTY | RGBA_FLAG_NEEDS_CLEAR);

[Moorviper]

why is it a typo??
Logical Or Operator = ||
Bitwise Or Operator = |

i think the logical should be used

[daym]

why is it a typo??

() is a bitwise operator (which does inversion). It makes little sense to mix bitwise () and logical (||) in that fashion.

Also, doing &(a|b) is a common idiom to clear the stated flags while &(a||b) is senseless (would always clear bit 0 - if a or b is not equal to 0, which is the case).

i think the logical should be used

Why do you think that?

[rellla]

daym seems to be right.
fixed with 1a41433