[FFmpeg-devel] [PATCH 2/9] avcodec/gdv: Replace divisions by shifts in rescale()

Michael Niedermayer michael at niedermayer.cc
Tue Sep 25 00:43:38 EEST 2018


On Mon, Aug 20, 2018 at 12:42:29AM +0200, Michael Niedermayer wrote:
> On Sun, Aug 05, 2018 at 11:55:06PM +0200, Nicolas George wrote:
> > Michael Niedermayer (2018-08-05):
> > > thats done in the next patch
> > 
> > Indeed, except for the part where the variable are turned to unsigned to
> > let the compiler optimize the division. Sorry to have missed that.
> 
> are you ok with leaving the shifts as in the patchset or should i redo the
> patchset with unsigned ?

CC-ing you so you dont miss this (though its not important)

if i hear nothing from anyone i will apply the patchset as is.
IIUC james is ok with that. 
But as said i can redo it with unsigned if people want.
Just dont want to spend time redoing a patchset when maybe everyone is fine
with it after the discussions. (its hard to read peoples preferance from
silence ...)

thx

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

You can kill me, but you cannot change the truth.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180924/a81010b8/attachment.sig>


More information about the ffmpeg-devel mailing list