[FFmpeg-devel] [PATCH 0/3] libavformat/protocols.c: avio_enum_protocols(): Cleanup

Nicolas George george at nsup.org
Wed Aug 11 22:12:38 EEST 2021


Michael Witten (12021-08-11):
> However, {2} is presented as a bunch of tiny little transformations
> that are intended to aid comprehension; they can be squashed into
> one commit as the maintainer sees fit (indeed, as shown below, the
> squashed diff is already quite comprehensible):

This is my my opinion, but better squash related changes: what matters
is the final code, how readable it is.

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/20210811/826a78aa/attachment.sig>


More information about the ffmpeg-devel mailing list