[FFmpeg-devel] discussion around the best way to patch the swscale for Blackfin
Fri Jun 22 13:51:51 CEST 2007
On 6/22/07, Reimar Doeffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de>
> On Thu, Jun 21, 2007 at 11:00:28PM -0400, Marc Hoffman wrote:
> > Does this look like a clean integration for the unscaled variant for the
> > swscale infrastructure? Seems reasonably clean to me but another set of
> > eyes would be great.
> Looks good, except
> 1) ff_bfin_get_unscaled_swscale (c); seems to be indented differently
> than the other code.
> 2) if (ENABLE_BFIN) ff_bfin_get_unscaled_swscale (c); avoids the #ifdef
> and matches current dsputil
On #1 I can't see that the indentation is incorrect. I must be missing
I'm actually not sure about this either do you mean this?
converts into the following which to me is a reasonable idea
if (flags & SWS_CPU_CAPS_BFIN)
but this would still need to be wrapped inside of an ifdef ARCH_BFIN because
the function is defined for compilation only bfin. Did I miss something?
More information about the ffmpeg-devel