On Wed, Aug 22, 2018 at 09:02:57PM +0200, Martin Vignali wrote:
> >
> > well then it should be ok
> > not sure if the ff_ prefix is ideal for a static inline function
> >
> >
> Do you think flt_2_uint16 is a better name ?
yes
thx
> (can probably be interesting to also add same kind of func for f
>
> well then it should be ok
> not sure if the ff_ prefix is ideal for a static inline function
>
>
Do you think flt_2_uint16 is a better name ?
(can probably be interesting to also add same kind of func for float to
uint8 in unscaled part)
Martin
___
f
On Wed, Aug 22, 2018 at 12:45:50PM +0200, Martin Vignali wrote:
> > > ---
> > > libswscale/input.c| 10 +-
> > > libswscale/swscale_internal.h | 20
> > > 2 files changed, 25 insertions(+), 5 deletions(-)
> >
> > please provide benchmark, what is the impact
> > ---
> > libswscale/input.c| 10 +-
> > libswscale/swscale_internal.h | 20
> > 2 files changed, 25 insertions(+), 5 deletions(-)
>
> please provide benchmark, what is the impact on speed from this ?
>
>
In my tests, the patch increase speed by around 20
On Mon, Aug 20, 2018 at 04:41:02PM +0200, Martin Vignali wrote:
> Hello,
>
> Patch in attach modify GrayFloat to Gray16 conversion
> using the same method currently use inside exr decoder (no float calc)
>
> duplicate the float_to_uint func inside swscale_internal
>
> Martin
> input.c
Hello,
Patch in attach modify GrayFloat to Gray16 conversion
using the same method currently use inside exr decoder (no float calc)
duplicate the float_to_uint func inside swscale_internal
Martin
0004-swscale-input-avoid-float-calc-for-grayFloat-to-uint.patch
Description: Binary data
_
