[FFmpeg-devel] [PATCH v2] avcodec/libjxlenc: support negative linesizes

[Leo Izen]

On 1/30/24 09:57, Andreas Rheinhardt wrote:
> 
> Please don't rely on the type of jxl_fmt.align here (which is out of our
> control). E.g. in the future it may be that libjxl supports only 32bit
> align values (i.e. uses uint32_t or so for it), but that we support
> 64bit (ptrdiff_t) linesizes and allocations, so that jxl_fmt.align *
> (info.ysize - 1) may overflow this even when -linesize fits into
> jxl_fmt.align.
> (Very unlikely given that align is size_t and they will likely not go
> back from this, but it could happen.)
> 

This would be an ABI break though, wouldn't it? Why do we need to work 
around a potential future ABI break?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature.asc
Type: application/pgp-signature
Size: 236 bytes
Desc: OpenPGP digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20240130/cf53c374/attachment.sig>