[FFmpeg-devel] [PATCH 01/23] lavc/movtextdec: fix ass header colors
Philip Langdale
philipl at overt.org
Thu Apr 9 19:43:27 EEST 2020
On Thu, 9 Apr 2020 15:09:45 +0000
John Stebbins <jstebbins at jetheaddev.com> wrote:
> >
>
> I missed this review earlier. What about the indentation is off? Do
> you mean the indent of the function parameters does not align after
> the opening paren? I was keeping the lines aligned without wrapping
> at 80 characters.
Yes, I believe he's referring to the function parameters and not the
overall indent. I would normally align to the paren. I think you can do
that and keep under 80 if you put each one on its own line?
--phil
More information about the ffmpeg-devel
mailing list