Re: [FFmpeg-devel] [PATCH 1/2] lavc/dvdsubdec: Move palette parsing to new function

2020-02-04 Thread Michael Kuron
I split off the documentation fix into a separate patch and sent the series of three patches yesterday. Carl Eugen Hoyos already merged the first one with the documentation fix. Can you take care of the remaining two patches? On Sun, Feb 2, 2020 at 5:08 PM Michael Niedermayer wrote: >> diff --gi

[FFmpeg-devel] [PATCH 1/3] doc: Fix typo for dvdsubdec

2020-02-03 Thread Michael Kuron
Signed-off-by: Michael Kuron --- doc/decoders.texi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/decoders.texi b/doc/decoders.texi index 0582b018b0..83515ae363 100644 --- a/doc/decoders.texi +++ b/doc/decoders.texi @@ -280,7 +280,7 @@ palette is stored in the IFO file

[FFmpeg-devel] [PATCH 2/3] lavc/dvdsubdec: Move palette parsing to new function

2020-02-03 Thread Michael Kuron
Signed-off-by: Michael Kuron --- libavcodec/Makefile| 1 + libavcodec/dvdsub.c| 33 + libavcodec/dvdsubdec.c | 22 ++ libavcodec/internal.h | 2 ++ 4 files changed, 42 insertions(+), 16 deletions(-) create mode 100644 libavcodec

[FFmpeg-devel] [PATCH 3/3] lavc/dvdsubenc: accept palette from options

2020-02-03 Thread Michael Kuron
Previously, the default palette would always be used. Now, we can accept a custom palette, just like dvdsubdec does. Signed-off-by: Michael Kuron --- doc/encoders.texi | 8 libavcodec/Makefile| 1 + libavcodec/dvdsubenc.c | 8 +++- 3 files changed, 16 insertions(+), 1

[FFmpeg-devel] [PATCH 2/2] lavc/dvdsubenc: accept palette from options

2020-02-01 Thread Michael Kuron
Previously, the default palette would always be used. Now, we can accept a custom palette, just like dvdsubdec does. Signed-off-by: Michael Kuron --- doc/encoders.texi | 8 libavcodec/Makefile| 1 + libavcodec/dvdsubenc.c | 8 +++- 3 files changed, 16 insertions(+), 1

[FFmpeg-devel] [PATCH 1/2] lavc/dvdsubdec: Move palette parsing to new function

2020-02-01 Thread Michael Kuron
Signed-off-by: Michael Kuron --- doc/decoders.texi | 2 +- libavcodec/Makefile| 1 + libavcodec/dvdsub.c| 33 + libavcodec/dvdsubdec.c | 22 ++ libavcodec/internal.h | 2 ++ 5 files changed, 43 insertions(+), 17 deletions

Re: [FFmpeg-devel] [PATCH] lavc/dvdsubenc: accept palette from options

2020-01-30 Thread Michael Kuron
Is there anything else you need me to change before this can be merged? If not, I am happy to split it into two patches if necessary. On Sat, Jan 25, 2020 at 3:33 PM Michael Kuron wrote: > > Previously, the default palette would always be used. > Now, we can accept a custom palette,

[FFmpeg-devel] [PATCH] lavc/dvdsubenc: accept palette from options

2020-01-25 Thread Michael Kuron
Previously, the default palette would always be used. Now, we can accept a custom palette, just like dvdsubdec does. Signed-off-by: Michael Kuron --- doc/decoders.texi | 2 +- doc/encoders.texi | 8 libavcodec/Makefile| 2 ++ libavcodec/dvdsub.c| 33

[FFmpeg-devel] [PATCH] lavc/dvdsubenc: accept palette from options

2020-01-25 Thread Michael Kuron
Previously, the default palette would always be used. Now, we can accept a custom palette, just like dvdsubdec does. Signed-off-by: Michael Kuron --- doc/decoders.texi | 2 +- doc/encoders.texi | 8 libavcodec/Makefile| 1 + libavcodec/dvdsub.c| 35

Re: [FFmpeg-devel] [PATCH] lavc/dvdsubenc: accept palette from options

2020-01-25 Thread Michael Kuron
On Sat, Jan 25, 2020 at 01:24 AM Carl Eugen Hoyos wrote: >> static const AVOption options[] = { >> +{"palette", "set the global palette", OFFSET(palette_str), >> AV_OPT_TYPE_STRING, { .str = NULL }, 0, 0, SE }, > > What happens if invalid values are passed as palette? The string is parsed un

[FFmpeg-devel] [PATCH] lavc/dvdsubenc: accept palette from options

2020-01-24 Thread Michael Kuron
Previously, the default palette would always be used. Now, we can accept a custom palette, just like dvdsubdec does. Signed-off-by: Michael Kuron --- doc/encoders.texi | 8 libavcodec/dvdsubenc.c | 19 ++- 2 files changed, 26 insertions(+), 1 deletion(-) diff

[FFmpeg-devel] [PATCH] lavc/dvdsubenc: accept palette from options

2020-01-24 Thread Michael Kuron
Previously, the default palette would always be used. Now, we can accept a custom palette, just like dvdsubdec does. Signed-off-by: Michael Kuron --- doc/encoders.texi | 9 + libavcodec/dvdsubenc.c | 19 ++- 2 files changed, 27 insertions(+), 1 deletion(-) diff

[FFmpeg-devel] [PATCH] lavc/dvdsubenc: accept palette from options

2020-01-24 Thread Michael Kuron
0001-lavc-dvdsubenc-accept-palette-from-options.patch Description: Binary data ___ 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...@ffmpe