On Sun, 16 Jul 2017 01:08:37 +0200
Luca Barbato <lu_z...@gentoo.org> wrote:
> On 7/14/17 7:14 PM, wm4 wrote:
> > + } else if (clear_size == 4) {
> > + uint32_t val = AV_RN32(clear);
> > + for (; dst_size >= 4; dst_size -= 4) {
> > + AV_WN32(dst, val);
> > + dst += 4;
> > + }
> > + }
>
> seems ok, my ocd would suggest to add also the 64bit variant, but... It
> seems good enough for now.
I'll add a 64 bit version, plus some other changes:
- simply the reduce-to-memset code
- use full range for alpha even for yuv (apparently that's how it works)
- allow passing 0 width/height, in which case the function is a NOP,
but the error return will indicate whether the pixfmt is supported
- change EINVAL to ENOSYS for unsupported pixfmt cases
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
