[FFmpeg-cvslog] r26289 - trunk/libavcodec/truemotion2.c

[Michael Niedermayer]

On Mon, Jan 10, 2011 at 06:56:35PM +0100, Reimar D?ffinger wrote:
> On Mon, Jan 10, 2011 at 02:58:49PM +0100, Michael Niedermayer wrote:
> > On Mon, Jan 10, 2011 at 08:12:57AM +0200, Kostya wrote:
> > > On Sun, Jan 09, 2011 at 09:41:52PM +0100, Stefano Sabatini wrote:
> > > [...]
> > > > 
> > > > People, plese try to use meaningful error codes, they help calling
> > > > code to detect and report which is the cause of the problem, -1
> > > > corresponds to EPERM which is not the correct reason most of the
> > > > times.
> > > 
> > > This idea has not been bikeshedded enough. But in general who cares what
> > > error was returned by decoder? -1 is just our way to say "generic error,
> > > we don't care about actual code"
> > 
> > agree
> 
> I understood the intial mail to mean that the main problem is that applications
> that try to make the value into an error string end up claiming the error was
> "permission denied", which can be really confusing.

I wonder if theres any code that does anything with these error codes
programatically reacting on them beyond EAGAIN seems hard
and printing a "whatever error" might look akward in addition and below
of a much more verbose av_log error message
is any application doing that?

Note, iam not at all saying the codes should not be improved ...


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Many things microsoft did are stupid, but not doing something just because
microsoft did it is even more stupid. If everything ms did were stupid they
would be bankrupt already.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20110111/6d3e34a0/attachment.pgp>