On Fri, Feb 10, 2012 at 10:53:13AM +0100, Diego Biurrun wrote: > On Thu, Feb 09, 2012 at 08:26:17PM -0500, Rafaël Carré wrote: > > --- > > AVCodecParserContext uses Parse_r_ but all other structures use Parse, > > so I guess the mp4v parser should too. > > > > libavcodec/mpeg4video_parser.c | 20 +++++++++++--------- > > 1 files changed, 11 insertions(+), 9 deletions(-) > > OK, I know I am a curmudgeon about log messages, but the subject of this > patch is not really precise. Parse_r_Context does not exist and "use" > is so generic it could mean anything. How about: > > mpeg4video parser: introduce struct Mp4vParseContext > > This obviates using ParseContext1, which is slated for removal.
Queued this and the rest of the set, substituting as log message mpegvideo/mpeg4video parser: move specific fields into private context This obviates using ParseContext1, which is slated for removal. which funman suggested on IRC. Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
