On Fri, May 11, 2012 at 10:35 AM, Mans Rullgard <[email protected]> wrote:
>
> These files all have pcm audio which is tested elsewhere.
>
> Signed-off-by: Mans Rullgard <[email protected]>
> ---
>  tests/fate/demux.mak             |    3 --
>  tests/fate/qt.mak                |    2 +-
>  tests/fate/screen.mak            |    4 +-
>  tests/fate/video.mak             |   25 +++++----
>  tests/ref/fate/8bps              |   24 ---------
>  tests/ref/fate/armovie-escape124 |    5 --
>  tests/ref/fate/bethsoft-vid      |   72 -------------------------
>  tests/ref/fate/bfi               |   58 --------------------
>  tests/ref/fate/corepng           |   20 -------
>  tests/ref/fate/creatureshock-avs |   37 -------------
>  tests/ref/fate/cyberia-c93       |    5 --
>  tests/ref/fate/dxtory            |    1 -
>  tests/ref/fate/id-cin-video      |   53 ------------------
>  tests/ref/fate/siff              |    4 --
>  tests/ref/fate/sp5x              |   10 ----
>  tests/ref/fate/tiertex-seq       |   78 ---------------------------
>  tests/ref/fate/tmv               |  111 
> --------------------------------------
>  tests/ref/fate/tscc-15bit        |   16 ------
>  tests/ref/fate/wc3movie-xan      |   36 -------------
>  19 files changed, 17 insertions(+), 547 deletions(-)
>

This seems to substantially lower our coverage for demuxers. In these
cases the code that demuxes the audio is still executed but now we no
longer get to see the timestamps or that we are grabbing the correct
bytes for each audio frame. What's the motivation for this? I under
stand trying to test features in isolation, but "decoding" linear PCM
audio isn't much more work than just copying, sometimes with a
byteswap.
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to