On Fri, Jun 26, 2020 at 12:50:40PM +0200, Nicolas George wrote: > lance.lmw...@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.
OK, I'll submit a patch to add extra \n for ERROR and WARNING message. I think it's OK to print one extra INFO \n if it's info level. > > Regards, > > -- > Nicolas George > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe". -- Thanks, Limin Wang _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
