[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