[FFmpeg-devel] [PATCH]lavc/g729dec: Support decoding ACELP.KELVIN

2019-08-19 Thread Carl Eugen Hoyos
Hi! Attached patch fixes tickets #4799 and #8081, thanks to Paul for his help. Please comment, Carl Eugen From 8af0b279ad0c25425d075498f60b0770528687a2 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Mon, 19 Aug 2019 23:34:37 +0200 Subject: [PATCH] lavc/g729dec: Support decoding Sipro ACEL

Re: [FFmpeg-devel] [PATCH]lavc/g729dec: Support decoding ACELP.KELVIN

2019-09-16 Thread Carl Eugen Hoyos
Am Fr., 23. Aug. 2019 um 09:14 Uhr schrieb Paul B Mahol : > > allcodecs.c entry is in wrong section, it should be audio one but yours is > in video section. > the avcodec.h entry may contradict with next libav entry so better move it > bellow hcom. Pushed with these changes. Thank you! Carl Euge

Re: [FFmpeg-devel] [PATCH]lavc/g729dec: Support decoding ACELP.KELVIN

2019-08-20 Thread Paul B Mahol
Do not use full caps name for short name. On Mon, Aug 19, 2019 at 11:38 PM Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes tickets #4799 and #8081, thanks to Paul for his help. > > Please comment, Carl Eugen > ___ > ffmpeg-devel mailing list > f

Re: [FFmpeg-devel] [PATCH]lavc/g729dec: Support decoding ACELP.KELVIN

2019-08-20 Thread Carl Eugen Hoyos
Am Di., 20. Aug. 2019 um 10:30 Uhr schrieb Paul B Mahol : > > Do not use full caps name for short name. New patch attached. Thank you, Carl Eugen From 8af0b279ad0c25425d075498f60b0770528687a2 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Mon, 19 Aug 2019 23:34:37 +0200 Subject: [PATCH] l

Re: [FFmpeg-devel] [PATCH]lavc/g729dec: Support decoding ACELP.KELVIN

2019-08-21 Thread Paul B Mahol
The configure line is not needed. Where is Makefile change? Changlelog one does not apply. On Tue, Aug 20, 2019 at 7:25 PM Carl Eugen Hoyos wrote: > Am Di., 20. Aug. 2019 um 10:30 Uhr schrieb Paul B Mahol >: > > > > Do not use full caps name for short name. > > New patch attached. > > Thank you

Re: [FFmpeg-devel] [PATCH]lavc/g729dec: Support decoding ACELP.KELVIN

2019-08-21 Thread Carl Eugen Hoyos
Am Mi., 21. Aug. 2019 um 10:40 Uhr schrieb Paul B Mahol : > > The configure line is not needed. Where is Makefile change? This line makes no sense. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmp

Re: [FFmpeg-devel] [PATCH]lavc/g729dec: Support decoding ACELP.KELVIN

2019-08-21 Thread Paul B Mahol
On Wed, Aug 21, 2019 at 10:44 AM Carl Eugen Hoyos wrote: > Am Mi., 21. Aug. 2019 um 10:40 Uhr schrieb Paul B Mahol >: > > > > The configure line is not needed. Where is Makefile change? > > This line makes no sense. > Nope, you do not make sense. Configure line is completely bollocks. Makefile

Re: [FFmpeg-devel] [PATCH]lavc/g729dec: Support decoding ACELP.KELVIN

2019-08-21 Thread Carl Eugen Hoyos
Am Mi., 21. Aug. 2019 um 10:54 Uhr schrieb Paul B Mahol : > > On Wed, Aug 21, 2019 at 10:44 AM Carl Eugen Hoyos > wrote: > > > Am Mi., 21. Aug. 2019 um 10:40 Uhr schrieb Paul B Mahol > > > > > The configure line is not needed. Where is Makefile change? > > > > This line makes no sense. > > Nope,

Re: [FFmpeg-devel] [PATCH]lavc/g729dec: Support decoding ACELP.KELVIN

2019-08-21 Thread Paul B Mahol
On Wed, Aug 21, 2019 at 11:12 AM Carl Eugen Hoyos wrote: > Am Mi., 21. Aug. 2019 um 10:54 Uhr schrieb Paul B Mahol >: > > > > On Wed, Aug 21, 2019 at 10:44 AM Carl Eugen Hoyos > > wrote: > > > > > Am Mi., 21. Aug. 2019 um 10:40 Uhr schrieb Paul B Mahol < > one...@gmail.com > > > > > > > > The c

Re: [FFmpeg-devel] [PATCH]lavc/g729dec: Support decoding ACELP.KELVIN

2019-08-21 Thread Jean-Baptiste Kempf
On Wed, Aug 21, 2019, at 11:35, Paul B Mahol wrote: > > > > > The configure line is not needed. Where is Makefile change? > > > > > > > > This line makes no sense. > > > > > > Nope, you do not make sense. > > > > I hope you agree it is difficult to "make sense" in an answer to > > something that ma

Re: [FFmpeg-devel] [PATCH]lavc/g729dec: Support decoding ACELP.KELVIN

2019-08-22 Thread Carl Eugen Hoyos
Am Mo., 19. Aug. 2019 um 23:37 Uhr schrieb Carl Eugen Hoyos : > > Hi! > > Attached patch fixes tickets #4799 and #8081, thanks to Paul for his help. New patch attached. Carl Eugen From 8af0b279ad0c25425d075498f60b0770528687a2 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Mon, 19 Aug 2019

Re: [FFmpeg-devel] [PATCH]lavc/g729dec: Support decoding ACELP.KELVIN

2019-08-23 Thread Paul B Mahol
allcodecs.c entry is in wrong section, it should be audio one but yours is in video section. the avcodec.h entry may contradict with next libav entry so better move it bellow hcom. Rest looks OK. On Thu, Aug 22, 2019 at 12:13 PM Carl Eugen Hoyos wrote: > Am Mo., 19. Aug. 2019 um 23:37 Uhr schri