On Fri, 8 May 2015 09:02:23 +0000 John Högberg <[email protected]> wrote:
> Hi again, > > I've changed my approach a bit to make it more general; instead of adding a > "carrier type" field to AVStream, I've added support for raising any event > through omission. Formats set `base_event_flags` whenever they create a > stream, > denoting which events will be raised unless explicitly cleared. To make use of > this you must call the new function av_reset_event_flags() to clear flags, but > the old method of "event_flags = 0" will still work perfectly fine for events > that are explicitly raised. > > I expect it won't break any existing code since NO_CARRIER is so far the only > event that may be raised in this manner, and any code that doesn't handle that > therefore doesn't need to use av_reset_event_flags() either. > > I'm hoping this is a step in the right direction. Thoughts? I think this is worse. Before, the API users was responsible for clearing the event flags, and suddenly a function has to be called? The previous approach seemed less intrusive too. _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
