On Mon, Sep 7, 2015 at 1:12 PM, Hendrik Leppkes <[email protected]> wrote: >> by that reasoning we should drop AV_PIX_FMT_FLAG_RGB and just do >> >> if (strstr(pix_fmt, "rgb") || strstr(pix_fmt, "bgr")) >> >> sometimes you need redundant data for consistency and for simpler >> interfaces, without having N different ways to access the same kind of >> information, in my opinion > > Now you're just being silly.
I was rather being sarcastic, it's silly to discuss over a simple 1bit flag. I can see the point that macros are "useful" to hide the complexity of the test, but an interface to check for this kind of thing already exists (flags) and the downside of public macros is that you need to document, maintain and make sure the new interfaces work. I insist that a single bitwise check is simpler to understand, use and support for users and devs alike. -- Vittorio _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
