[Ffmpeg-devel] [PATCH] dct-test remap emms to emms_c

[Måns Rullgård]

Marc Hoffman <mmh at pleasantst.com> writes:

> M?ns Rullg?rd writes:
>  > Marc Hoffman <mmh at pleasantst.com> writes:
>  > 
>  > > svn ci -m'remapping emms to emms_c, to allow compilation on non x86 platforms'
>  > 
>  > That's a very odd description of your change.  And please just write
>  > the text without svn ci in front of it.  All committers know the svn
>  > command syntax.
>
> eliminating invocation to deprecated emms using emms_c instead

How about something simple, such as "replace emms() with emms_c()"?

> Index: dct-test.c
> ===================================================================
> --- dct-test.c	(revision 8564)
> +++ dct-test.c	(working copy)
> @@ -184,7 +185,7 @@
>  #endif
>  
>          fdct_func(block);
> -        emms(); /* for ff_mmx_idct */
> +        emms_c(); /* for ff_mmx_idct */
>  
>          if (fdct_func == fdct_ifast
>  #ifndef FAAN_POSTSCALE
> @@ -286,7 +287,7 @@
>          it1 += NB_ITS_SPEED;
>          ti1 = gettime() - ti;
>      } while (ti1 < 1000000);
> -    emms();
> +    emms_c();
>  
>      printf("%s %s: %0.1f kdct/s\n",
>             is_idct ? "IDCT" : "DCT",
> @@ -446,7 +447,7 @@
>          it1 += NB_ITS_SPEED;
>          ti1 = gettime() - ti;
>      } while (ti1 < 1000000);
> -    emms();
> +    emms_c();
>  
>      printf("%s %s: %0.1f kdct/s\n",
>             1 ? "IDCT248" : "DCT248",

-- 
M?ns Rullg?rd
mans at mansr.com