On 05/05/16 01:29, Diego Biurrun wrote:
> On Wed, May 04, 2016 at 09:15:33PM +0900, Luca Barbato wrote:
>> On 04/05/16 16:38, Diego Biurrun wrote:
>>> -
>>> -        av_log(ctx, AV_LOG_VERBOSE,
>>> -               "idx:%d name:'%s' type:%s explanation:'%s' ",
>>> -               i, info.name,
>>> -               info.type == F0R_PARAM_BOOL     ? "bool"     :
>>> -               info.type == F0R_PARAM_DOUBLE   ? "double"   :
>>> -               info.type == F0R_PARAM_COLOR    ? "color"    :
>>> -               info.type == F0R_PARAM_POSITION ? "position" :
>>> -               info.type == F0R_PARAM_STRING   ? "string"   : "unknown",
>>> -               info.explanation);
>>> -
>>
>> This looks stray.
> 
> I dropped it on purpose.  It's all trivial debug output, but just half
> of it is under DEBUG ifdefs and only the latter part is broken.  I can
> keep the non-DEBUG debug output if you prefer.

Keep it or change the subject to "drop the overly verbose output".

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to