[FFmpeg-devel] [PATCH V5 1/2] avutil: add ROI (Region Of Interest) data struct and bump version

Nicolas George george at nsup.org
Sat Jan 5 06:51:27 EET 2019


James Almer (12019-01-04):
> We already did size_t for AVSphericalMapping and had to change them to
> uint32_t.
> size_t is not the correct type at all, as video dimensions are not
> system dependent,

I fully agree with that.

>		    and it will generate the same issues we already
> experienced with AVSphericalMapping in fate tests.

I disagree with that: for some things, size_t would be the correct type,
and in that case, the FATE test should work. A misdesigned FATE test
should not direct the choice of type. Also, IIUC, somebody said the
issue was fixed.

For me, the definite argument is: All the rest of the code uses int for
frame size and coordinate, so new code should use int too unless there
is a very good and specific reason.

Note that while we disagree on the reasons, I think we agree on the
conclusion, which is what matter.

Regards,

-- 
  Nicolas George


More information about the ffmpeg-devel mailing list