<html> <head> <base href="https://bugs.freedesktop.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW - _cairo_surface_is_snapshot() return error result on armcc on RVDS/MDK" href="https://bugs.freedesktop.org/show_bug.cgi?id=93281#c3">Comment # 3</a> on <a class="bz_bug_link bz_status_NEW " title="NEW - _cairo_surface_is_snapshot() return error result on armcc on RVDS/MDK" href="https://bugs.freedesktop.org/show_bug.cgi?id=93281">bug 93281</a> from <a class="email" href="mailto:b.harrington@samsung.com" title="Bryce Harrington <b.harrington@samsung.com>"> Bryce Harrington</a> <pre>Setting something like CAIRO_SURFACE_TYPE_IMAGE = 0x0000 to force the type looks kind of kludgy to me. Offhand I think Bill's right that the tested value should be added to the enum. In general casting enums from one to another seems crude; there's no way to be sure that one day someone adds FOO as item 42 in one enum definition and someone else adds BAR as 42 in the other, which mean completely different and incompatible things? If there's just a handful of enum casts, I'd say just fix them to work without needing casts. If this is used extensively throughout the codebase, this probably needs further thought.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the QA Contact for the bug.</li> </ul> </body> </html>