[FFmpeg-devel] [PATCH V5 1/2] avutil: add ROI (Region Of Interest) data struct and bump version
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
> 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.
More information about the ffmpeg-devel