[FFmpeg-devel] [PATCH] libavcodec/ccaption_dec.c: Fixed indentation overwriting text in the cea608 caption decoder.

2021-01-26 Thread levi
From: Levi Dooley There was an assumption in the existing code that indentation would not occur more than once on the same row. This was a bad assumption. There are examples of 608 streams which call handle_pac multiple times on the same row with different indentation. As the code was before

[FFmpeg-devel] [PATCH] libavcodec/ccaption_dec.c: Fixed indentation overwriting text in the cea608 caption decoder.

2021-01-27 Thread levi
From: Levi Dooley There was an assumption in the existing code that indentation would not occur more than once on the same row. This was a bad assumption. There are examples of 608 streams which call handle_pac multiple times on the same row with different indentation. As the code was before

[FFmpeg-devel] [PATCH] libavcodec/ccaption_dec.c: Fixed indentation overwriting text in the cea608 caption decoder.

2021-01-25 Thread Levi Dooley
x27;t end up between two words. Signed-off-by: Levi Dooley Here's a link to a sample file that will reproduce this issue. https://snapstream-dev-test-public.s3.us-east-1.amazonaws.com/ffmpeg-caption-issue/cleveland-clip.ts The issue can be reproduced by running the following command: > ff

Re: [FFmpeg-devel] [PATCH] libavcodec/ccaption_dec.c: Fixed indentation overwriting text in the cea608 caption decoder.

2021-01-27 Thread Levi Dooley
g the test failures locally. The test failures did point out a bug in the patch, and I am fixing it now. I will submit an updated patch as soon as that is done. Thanks, Levi On Wed, Jan 27, 2021 at 3:19 PM Andreas Rheinhardt < andreas.rheinha...@gmail.com> wrote: > Aman Karmani: > >