Re: [FFmpeg-devel] [PATCH] avcodec/codec_desc: rename smv video codec name to smvjpeg to match decoder name

2015-06-29 Thread Reimar Döffinger
On Tue, Jun 16, 2015 at 09:18:13PM +0200, Hendrik Leppkes wrote: > On Tue, Jun 16, 2015 at 8:33 PM, wm4 wrote: > > On Tue, 16 Jun 2015 13:29:55 + > > Paul B Mahol wrote: > > > >> Also fixes clash with smv audio codec. > >> > >> Signed-off-by: Paul B Mahol > >> --- > >> libavcodec/codec_desc

Re: [FFmpeg-devel] [PATCH] avcodec/codec_desc: rename smv video codec name to smvjpeg to match decoder name

2015-06-16 Thread Andreas Cadhalpun
On 16.06.2015 22:29, Paul B Mahol wrote: > On 6/16/15, Andreas Cadhalpun wrote: >> On 16.06.2015 21:31, Hendrik Leppkes wrote: >>> I see no reason these strings should be considered API. They are >>> descriptive names, not option tokens or anything. >>> If you want a reliable static way to identif

Re: [FFmpeg-devel] [PATCH] avcodec/codec_desc: rename smv video codec name to smvjpeg to match decoder name

2015-06-16 Thread Paul B Mahol
On 6/16/15, Andreas Cadhalpun wrote: > On 16.06.2015 21:31, Hendrik Leppkes wrote: >> I see no reason these strings should be considered API. They are >> descriptive names, not option tokens or anything. >> If you want a reliable static way to identify a codec, use the id. > > At least currently u

Re: [FFmpeg-devel] [PATCH] avcodec/codec_desc: rename smv video codec name to smvjpeg to match decoder name

2015-06-16 Thread Andreas Cadhalpun
On 16.06.2015 21:31, Hendrik Leppkes wrote: > I see no reason these strings should be considered API. They are > descriptive names, not option tokens or anything. > If you want a reliable static way to identify a codec, use the id. At least currently using the codec id is the only way to reliably

Re: [FFmpeg-devel] [PATCH] avcodec/codec_desc: rename smv video codec name to smvjpeg to match decoder name

2015-06-16 Thread wm4
On Tue, 16 Jun 2015 21:55:13 +0200 Hendrik Leppkes wrote: > On Tue, Jun 16, 2015 at 9:45 PM, wm4 wrote: > > On Tue, 16 Jun 2015 21:31:26 +0200 > > Hendrik Leppkes wrote: > > > >> On Tue, Jun 16, 2015 at 9:23 PM, wm4 wrote: > >> > On Tue, 16 Jun 2015 21:18:13 +0200 > >> > Hendrik Leppkes wrote

Re: [FFmpeg-devel] [PATCH] avcodec/codec_desc: rename smv video codec name to smvjpeg to match decoder name

2015-06-16 Thread Hendrik Leppkes
On Tue, Jun 16, 2015 at 9:45 PM, wm4 wrote: > On Tue, 16 Jun 2015 21:31:26 +0200 > Hendrik Leppkes wrote: > >> On Tue, Jun 16, 2015 at 9:23 PM, wm4 wrote: >> > On Tue, 16 Jun 2015 21:18:13 +0200 >> > Hendrik Leppkes wrote: >> > >> >> On Tue, Jun 16, 2015 at 8:33 PM, wm4 wrote: >> >> > On Tue,

Re: [FFmpeg-devel] [PATCH] avcodec/codec_desc: rename smv video codec name to smvjpeg to match decoder name

2015-06-16 Thread wm4
On Tue, 16 Jun 2015 21:31:26 +0200 Hendrik Leppkes wrote: > On Tue, Jun 16, 2015 at 9:23 PM, wm4 wrote: > > On Tue, 16 Jun 2015 21:18:13 +0200 > > Hendrik Leppkes wrote: > > > >> On Tue, Jun 16, 2015 at 8:33 PM, wm4 wrote: > >> > On Tue, 16 Jun 2015 13:29:55 + > >> > Paul B Mahol wrote: >

Re: [FFmpeg-devel] [PATCH] avcodec/codec_desc: rename smv video codec name to smvjpeg to match decoder name

2015-06-16 Thread Hendrik Leppkes
On Tue, Jun 16, 2015 at 9:23 PM, wm4 wrote: > On Tue, 16 Jun 2015 21:18:13 +0200 > Hendrik Leppkes wrote: > >> On Tue, Jun 16, 2015 at 8:33 PM, wm4 wrote: >> > On Tue, 16 Jun 2015 13:29:55 + >> > Paul B Mahol wrote: >> > >> >> Also fixes clash with smv audio codec. >> >> >> >> Signed-off-by

Re: [FFmpeg-devel] [PATCH] avcodec/codec_desc: rename smv video codec name to smvjpeg to match decoder name

2015-06-16 Thread wm4
On Tue, 16 Jun 2015 21:18:13 +0200 Hendrik Leppkes wrote: > On Tue, Jun 16, 2015 at 8:33 PM, wm4 wrote: > > On Tue, 16 Jun 2015 13:29:55 + > > Paul B Mahol wrote: > > > >> Also fixes clash with smv audio codec. > >> > >> Signed-off-by: Paul B Mahol > >> --- > >> libavcodec/codec_desc.c |

Re: [FFmpeg-devel] [PATCH] avcodec/codec_desc: rename smv video codec name to smvjpeg to match decoder name

2015-06-16 Thread Hendrik Leppkes
On Tue, Jun 16, 2015 at 8:33 PM, wm4 wrote: > On Tue, 16 Jun 2015 13:29:55 + > Paul B Mahol wrote: > >> Also fixes clash with smv audio codec. >> >> Signed-off-by: Paul B Mahol >> --- >> libavcodec/codec_desc.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/liba

Re: [FFmpeg-devel] [PATCH] avcodec/codec_desc: rename smv video codec name to smvjpeg to match decoder name

2015-06-16 Thread wm4
On Tue, 16 Jun 2015 13:29:55 + Paul B Mahol wrote: > Also fixes clash with smv audio codec. > > Signed-off-by: Paul B Mahol > --- > libavcodec/codec_desc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/codec_desc.c b/libavcodec/codec_desc.c > index c1

Re: [FFmpeg-devel] [PATCH] avcodec/codec_desc: rename smv video codec name to smvjpeg to match decoder name

2015-06-16 Thread Michael Niedermayer
On Tue, Jun 16, 2015 at 01:29:55PM +, Paul B Mahol wrote: > Also fixes clash with smv audio codec. > > Signed-off-by: Paul B Mahol > --- > libavcodec/codec_desc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) LGTM thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730

[FFmpeg-devel] [PATCH] avcodec/codec_desc: rename smv video codec name to smvjpeg to match decoder name

2015-06-16 Thread Paul B Mahol
Also fixes clash with smv audio codec. Signed-off-by: Paul B Mahol --- libavcodec/codec_desc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/codec_desc.c b/libavcodec/codec_desc.c index c1694f3..f32843a 100644 --- a/libavcodec/codec_desc.c +++ b/libavcodec/codec_