Hi, On Mon, Apr 25, 2011 at 10:46 PM, Diego Biurrun <[email protected]> wrote: > On Tue, Apr 26, 2011 at 12:27:48AM +0200, Janne Grunau wrote: >> --- >> libswscale/utils.c | 10 +++++----- >> 1 files changed, 5 insertions(+), 5 deletions(-) > > OK > >> --- a/libswscale/utils.c >> +++ b/libswscale/utils.c >> @@ -817,11 +817,11 @@ int sws_init_context(SwsContext *c, SwsFilter >> *srcFilter, SwsFilter *dstFilter) >> >> if (!isSupportedIn(srcFormat)) { >> - av_log(NULL, AV_LOG_ERROR, "swScaler: %s is not supported as input >> pixel format\n", sws_format_name(srcFormat)); >> + av_log(c, AV_LOG_ERROR, "%s is not supported as input pixel >> format\n", sws_format_name(srcFormat)); > > extra good karma for breaking long lines
We had a rule for this: A) no new patches necessary for these kind of comments, and B) you can fix it yourself. Let's try to keep stuff moving, I see a lot of nitpick-style comments again and they're not always as helpful as they're intended... Ronald _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
