On Mon, 2020-04-06 at 20:03 +0200, Nicolas George wrote:
> John Stebbins (12020-04-06):
> > A conversion from rgb to bgr is necessary
> > ---
> > libavcodec/movtextdec.c | 11 +++++++----
> > 1 file changed, 7 insertions(+), 4 deletions(-)
> >
> > diff --git a/libavcodec/movtextdec.c b/libavcodec/movtextdec.c
> > index c38c5edce6..05becaf64d 100644
> > --- a/libavcodec/movtextdec.c
> > +++ b/libavcodec/movtextdec.c
> > @@ -48,6 +48,8 @@
> > #define TOP_CENTER 8
> > #define TOP_RIGHT 9
> >
> > +#define RGB_TO_BGR(c) (((c) & 0xff) << 16 | ((c) & 0xff00) | (((c)
> > >> 16) & 0xff))
> > +
> > typedef struct {
> > char *font;
> > int fontsize;
> > @@ -448,10 +450,11 @@ static int mov_text_init(AVCodecContext
> > *avctx) {
> > MovTextContext *m = avctx->priv_data;
> > ret = mov_text_tx3g(avctx, m);
> > if (ret == 0) {
> > - return ff_ass_subtitle_header(avctx, m->d.font, m-
> > >d.fontsize, m->d.color,
> > - m->d.back_color, m->d.bold, m-
> > >d.italic,
> > - m->d.underline,
> > ASS_DEFAULT_BORDERSTYLE,
> > - m->d.alignment);
> > + return ff_ass_subtitle_header(avctx, m->d.font, m-
> > >d.fontsize,
> > + RGB_TO_BGR(m->d.color),
> > + RGB_TO_BGR(m->d.back_color),
> > + m->d.bold, m->d.italic, m->d.underline,
> > + ASS_DEFAULT_BORDERSTYLE, m->d.alignment);
>
> Indentation is off. It was off before, but since the lines are
> changed
> it should be ok now.
>
> Can't judge on semantic.
>
> > } else
> > return ff_ass_subtitle_header_default(avctx);
> > }
>
>
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.
_______________________________________________
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".
