On Sun, Jul 13, 2014 at 11:47:06PM +0200, Katerina Barone-Adesi wrote:
> New tests: fate-ra3-144 for RealAudio 1.0
>            fate-ra4-28_8 for RealAudio 2.0 containing the 28.8 codec.
> 
> New targets: fate-real, fate-realaudio
> ---
>  tests/fate/real.mak     | 13 +++++++--
>  tests/ref/fate/ra3-144  | 51 ++++++++++++++++++++++++++++++++++
>  tests/ref/fate/ra4-28_8 | 73 
> +++++++++++++++++++++++++++++++++++++++++++++++++
>  3 files changed, 135 insertions(+), 2 deletions(-)
>  create mode 100644 tests/ref/fate/ra3-144
>  create mode 100644 tests/ref/fate/ra4-28_8

fate: Add tests for RealAudio 14.4 and 28.8

> --- a/tests/fate/real.mak
> +++ b/tests/fate/real.mak
> @@ -1,3 +1,11 @@
> +# NOTE: the REALAUDIO tests need to use the RA demuxer once
> +# it is available
> +FATE_SAMPLES_REALAUDIO-$(call DEMDEC, RM, RA_144) += fate-ra3-144
> +fate-ra3-144: CMD = framecrc -i $(TARGET_SAMPLES)/realaudio/ra3.ra
> +
> +FATE_SAMPLES_REALAUDIO-$(call DEMDEC, RM, RA_288) += fate-ra4-28_8
> +fate-ra4-28_8: CMD = framecrc -i $(TARGET_SAMPLES)/realaudio/ra4-28_8.ra

Just FATE_REALAUDIO is enough.  I'd drop the _ from 28_8 for consistency
with most of the rest of the codebase.

> @@ -38,9 +46,10 @@ $(FATE_SIPR): CMP = oneoff
>  
> -FATE_SAMPLES_AVCONV += $(FATE_SAMPLES_REALMEDIA_AUDIO-yes) 
> $(FATE_SAMPLES_REALMEDIA_VIDEO-yes)
> -
> +FATE_SAMPLES_AVCONV += $(FATE_SAMPLES_REALMEDIA_AUDIO-yes) 
> $(FATE_SAMPLES_REALMEDIA_VIDEO-yes) $(FATE_SAMPLES_REALAUDIO-yes)

nit: alphabetical order

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to