[FFmpeg-devel] [PATCH 2/2] dsputil/me/pixels: Actually use av_restrict

Michael Niedermayer michaelni at gmx.at
Sat Sep 8 17:33:36 CEST 2012


On Sat, Sep 08, 2012 at 08:35:39AM -0400, Derek Buitenhuis wrote:
> On 07/09/2012 4:19 PM, Michael Niedermayer wrote:
> > i wonder, isnt it simpler and cleaner if these get replaced by that
> > msvc wraper script ?
> > or was there some other platform/case that needs these ?
> 
> I think there is a tentative patch for this in the converter somewhere. I think it
> is intended to make it in.
> 
> My rationale is: why bother having av_restrict and not use it where it helps?

sure if it helps lets use it, but does it for the following:
 libavcodec/alpha/dsputil_alpha.c    |    4 ++--
 libavcodec/alpha/dsputil_alpha.h    |    4 ++--
 libavcodec/alpha/motion_est_alpha.c |    4 ++--
 libavcodec/arm/dsputil_init_armv6.c |    2 +-
 libavcodec/bfin/dsputil_bfin.h      |    2 +-
 libavcodec/bfin/pixels_bfin.S       |    2 +-
 libavcodec/ppc/dsputil_altivec.c    |    4 ++--

?

i mean alpha/arm/bfin/ppc arent MSVC targets AFAIK

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

Opposition brings concord. Out of discord comes the fairest harmony.
-- Heraclitus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120908/e459fc99/attachment.asc>


More information about the ffmpeg-devel mailing list