[FFmpeg-devel] [PATCH v13 2/3] avfilter/vf_showinfo: display H.26[45] user data unregistered sei message

[Nicolas George]

lance.lmwang at gmail.com (12020-06-26):
> Yes, It make sense. Now there are ERROR, INFO, WARNING. Can we change all of the
> to INFO? If can't, then we'll had to remove the end \n and add for every case.

Absolutely not. ERRORs need to be visible even if INFO is hidden.

Personally, I consider that the messages at different log levels are
just completely separate, and should not be interleaved within a single
line. I can accept tidbits of VERBOSE within a line of INFO, but that's
the maximum.

In this case, that means the ERROR message needs to be complete, and the
INFO needs to be consistent even if there is an error.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20200626/dcf7b4c7/attachment.sig>