[FFmpeg-devel] [PATCH] lavu/error: use a sorted table for storing error codes and strings, add test

Stefano Sabatini stefasab at gmail.com
Fri Jun 1 23:46:39 CEST 2012


On date Monday 2012-02-06 20:04:09 +0100, Stefano Sabatini encoded:
> On date Monday 2012-02-06 19:56:08 +0100, Reimar Döffinger encoded:
> > On 6 Feb 2012, at 16:20, Stefano Sabatini <stefasab at gmail.com> wrote:
> > 
> > > On date Monday 2012-02-06 14:57:05 +0100, Nicolas George encoded:
> > >> L'octidi 18 pluviôse, an CCXX, Stefano Sabatini a écrit :
> > >>> The table is mostly useful for enumerating the available AVERROR* in the
> > >>> test output.
> > >> 
> > >> The idea is interesting, but I find that keeping the table ordered by tag is
> > >> rather bad for readability and maintainability. The table is quite small: a
> > >> simple linear search could be efficient enough.
> > > 
> > > Updated, output is now:
> > 
> 
> > Why did you leave the comment about it being required to be sorted in?
> 
> Doh, fixed locally.

Updated.

Now I'm seeing if it would be possible to implement an av_err2str()
macro (maybe using the same trick of av_time2str()?).
-- 
FFmpeg = Fundamental and Forgiving Multipurpose Powered Elected Guru
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-lavu-error-use-a-table-for-storing-error-codes-and-s.patch
Type: text/x-diff
Size: 5986 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120601/2863b134/attachment.bin>


More information about the ffmpeg-devel mailing list