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

[Nicolas George]

James Almer (12020-07-10):
> It's adding an extra parameter to get a value that ultimately can be
> derived from the output of another parameter. If you can use the >0 part
> of the return value for that,

Yes. And for me, it is totally a good thing.

So why are you calling it ugly?

This discussion feels to me like "Why is planting trees a bad thing? -
It makes the neighborhood greener."

Ugly is using the same channel to carry to completely different pieces
of information. And extra parameters are not a bad thing.

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/20200710/f1210675/attachment.sig>