[FFmpeg-devel] [PATCH] alsdec: Consistently use an exclamation mark for error messages.

Stefano Sabatini stefasab at gmail.com
Sat Jan 5 22:35:00 CET 2013


On date Saturday 2013-01-05 17:54:19 +0100, Thilo Borgmann encoded:
> Am 05.01.13 16:54, schrieb Stefano Sabatini:
> > On date Saturday 2013-01-05 15:53:07 +0100, Thilo Borgmann encoded:
> >> For consistency...
> >>
> >> -Thilo
> > 
> >> From 67d5795b8fc0ffc6eccffc45bca22e9303a4fe8e Mon Sep 17 00:00:00 2001
> >> From: Thilo Borgmann <thilo.borgmann at googlemail.com>
> >> Date: Sat, 5 Jan 2013 15:38:06 +0100
> >> Subject: [PATCH] alsdec: Consistently use an exclamation mark for error messages.
> > 
> > Why this?!
> > 
> > Seriously, we shouldn't scary users in case of errors! Also it is not
> > very consistent with the rest of the codebase error messages! Did you
> > see it?!
> 
> I see it is consistent within the als decoder what has never been complained
> about. These are some error messages where no ! has been in use yet, there have
> already been more with a ! in use.
> 
> I don't see how a ! scares someone. An error should not go unnoticed and that is
> what a ! shall express. Yes, this is enough content for a looong discussion though.
> 
> I would like to follow any global log-message rule we have but I'm not aware of
> any, have we? So all I can do is to keep my littly maintainance universe as
> pretty as possible.

Man, do as you prefer, it just reads a bit strange to see "!"  at the
end of an error message, and it activates a subconscious alarm bell in
the reader, not that I'm saying that is important, just that I'd
rather remove all the "!" which seems more consistent with the rest of
the codebase.
-- 
FFmpeg = Frightening & Forgiving Maxi Peaceful Excellent Gospel


More information about the ffmpeg-devel mailing list