On 2/27/2025 7:53 PM, Vittorio Giovara wrote:
On Thu, Feb 27, 2025 at 11:47 PM <epira...@gmail.com> wrote:On 27 Feb 2025, at 2:10, Michael Niedermayer wrote:Signed-off-by: Michael Niedermayer <mich...@niedermayer.cc> --- doc/developer.texi | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/doc/developer.texi b/doc/developer.texi index a1bfe180c9b..6a753f99da6 100644 --- a/doc/developer.texi +++ b/doc/developer.texi @@ -179,18 +179,17 @@ int fields = ilace ? 2 : 1; @end example @item -No braces around single-line blocks: +No braces around single-line blocks, unless they are followed by anelse (to keep future patches cleaner)I agree with this, however people in the past pointed it out when new code did this, IIRC. So if we merge this, people should stop flagging this in reviews, else its frustrating for new contributors following the documentation and then getting contradicting reviews.would loooove a format defining script with a prehook that formats your patches before sending <3
That can be done automatically as one of the many jobs CI runs once we move to forgejo/gitlab. Same with every other check in patcheck.
OpenPGP_signature.asc
Description: OpenPGP digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
