[FFmpeg-devel] [PATCH] ff_shift (left shift with allowed negative offset) implementation
Sun May 11 13:43:12 CEST 2008
On Sun, May 11, 2008 at 09:45:56AM +0700, Vladimir Voroshilov wrote:
> 2008/5/11 Michael Niedermayer <michaelni at gmx.at>:
> > On Sat, May 10, 2008 at 11:49:25PM +0700, Vladimir Voroshilov wrote:
> >> 2008/5/10 Michael Niedermayer <michaelni at gmx.at>:
> >> > On Sat, May 10, 2008 at 08:42:09PM +0700, Vladimir Voroshilov wrote:
> >> >> 2008/5/10, Michael Niedermayer <michaelni at gmx.at>:
> >> [...]
> >> >> > and the name is not good, it should give some hint that offset is meant
> >> >> > leftward and that value is signed.
> >> >>
> >> >> the given name is the best a can invent for it.
> >> >> Please anybody suggest better.
> >> >
> >> > ff_bidir_sal makes sense to me but iam not sure if it makes sense to anyone
> >> > else (ones not knowing x86 asm), sal being shift arithmetic (signed) left.
> >> >
> >> > maybe someone else has a better suggestion
> >> ff_shl_bidir ?
> > shl == unsigned shift to me
> Then i don't see anything better than yours.
> +static inline int ff_bidir_sal(int value, int offset)
as this is static it actually doesnt need the ff_ prefix
and patch ok besides this.
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Complexity theory is the science of finding the exact solution to an
approximation. Benchmarking OTOH is finding an approximation of the exact
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-devel