[FFmpeg-devel] [PATCH v2 1/3] libavutil/imgutils: add utility to get plane sizes

[Nicolas George]

James Almer (12020-07-20):
> No, i'll push v3 soon if my argumentation below was not enough to
> convince Nicolas or Michael. My intention is to use ints for the new
> function, not to postpone committing it in any form indefinitely.

Sorry, I missed you mail earlier, I only read your arguments nowish.

You are emphasizing that the "future-proof" argument is rather weak,
which I was already aware. And you are underplaying the fact that it
belongs to a trend to always delay necessary changes.

Anyway, even if all the arguments for using the proper types are all
very weak, there are several, and together I am still convinced they
exceed "consistency" easily.

consistently good > inconsistently good and bad > consistently bad
                                                ^
                                                |
             this is the discussion we are having

Or, as John Oliver pointed last Sunday: just because you reopened
restaurants does not mean you have to reopen schools too for the sake of
consistency.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20200722/335c658b/attachment.sig>