Re: [FFmpeg-devel] [PATCH 1/3] vidstab*: Remove accidentally exported av_2_vs_pixel_format()

2014-08-16 Thread James Almer
On 16/08/14 9:47 PM, Timothy Gu wrote: > Also correctly namespace other functions in vidstabutils, and decrease > difference from Libav. > > Initial-patch-by: Vittorio Giovara > Signed-off-by: Timothy Gu > --- > > Does this need an FF_API_VIDSTAB? > > doc/APIchanges| 4 +++

Re: [FFmpeg-devel] Invitation to VDD and registration

2014-08-16 Thread Jean-Baptiste Kempf
On 20 Jul, Jean-Baptiste Kempf wrote : > I'd like to invite you to the VideoLAN Dev Days 2014, the 3rd week-end > of September, in Dublin, Ireland. Google is providing the hosting. I would like to remind that you should register before August 20, if you want to be sponsored. In any cases, you sho

[FFmpeg-devel] [PATCH 2/3] vidstabutils: improve documentation

2014-08-16 Thread Timothy Gu
Signed-off-by: Timothy Gu --- libavfilter/vidstabutils.h | 17 ++--- 1 file changed, 14 insertions(+), 3 deletions(-) diff --git a/libavfilter/vidstabutils.h b/libavfilter/vidstabutils.h index 93278f6..c6d6ced 100644 --- a/libavfilter/vidstabutils.h +++ b/libavfilter/vidstabutils.h @

[FFmpeg-devel] [PATCH 3/3] vidstabutils: fix indentation

2014-08-16 Thread Timothy Gu
Signed-off-by: Timothy Gu --- libavfilter/vidstabutils.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/libavfilter/vidstabutils.c b/libavfilter/vidstabutils.c index 1f9f331..13544cf 100644 --- a/libavfilter/vidstabutils.c +++ b/libavfilter/vidstabutils.c @@ -52,11

[FFmpeg-devel] [PATCH 1/3] vidstab*: Remove accidentally exported av_2_vs_pixel_format()

2014-08-16 Thread Timothy Gu
Also correctly namespace other functions in vidstabutils, and decrease difference from Libav. Initial-patch-by: Vittorio Giovara Signed-off-by: Timothy Gu --- Does this need an FF_API_VIDSTAB? doc/APIchanges| 4 libavfilter/version.h | 2 +- libavfilter/vf

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-16 Thread Russ Allbery
Derek Buitenhuis writes: > On 8/16/2014 11:27 PM, wm4 wrote: >> That is very valuable advice. We'll get to work right away. > I've added it to my TODO: > * Don't write bugs. That's a really bad way of avoiding security bugs. I'm sure you know that and are just being flippant, but I have to sa

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-16 Thread Russ Allbery
wm4 writes: > Russ Allbery wrote: >> Note that all of the above statements also apply to libav. As near as I >> can tell, this is not a distinguishing characteristic between the two >> projects. > And that's an argument against switching to FFmpeg exactly how? It's not. Nor was I trying to m

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-16 Thread Andreas Cadhalpun
Hi, On 16.08.2014 17:49, Pau Garcia i Quiles wrote: On Sat, Aug 16, 2014 at 5:30 PM, Nicolas George mailto:geo...@nsup.org>> wrote: The only option is to make sure the users do not suffer from the fork, by making sure they can easily use the version that is most suited for thei

Re: [FFmpeg-devel] [PATCH] ffprobe: add -show_data_hash option.

2014-08-16 Thread Michael Niedermayer
On Sun, May 04, 2014 at 04:39:01PM +0200, Nicolas George wrote: > Signed-off-by: Nicolas George > --- > doc/ffprobe.texi | 4 > doc/ffprobe.xsd | 2 ++ > ffprobe.c| 39 +++ > 3 files changed, 45 insertions(+) > > > This time with a list of cip

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-16 Thread Derek Buitenhuis
On 8/16/2014 11:27 PM, wm4 wrote: > That is very valuable advice. We'll get to work right away. I've added it to my TODO: * Don't write bugs. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-16 Thread wm4
On Sat, 16 Aug 2014 11:59:20 -0700 Russ Allbery wrote: > The problem, however, is that taking security seriously, while possibly > necessary, is not sufficient. I'm glad that FFmpeg takes security > seriously, but what FFmpeg needs is to *have fewer security bugs*. That is very valuable advice.

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-16 Thread compn
On Sat, 16 Aug 2014 11:59:20 -0700 Russ Allbery wrote: > Two, the security team has already said that FFmpeg is not just "one > more package," and that no, they can't handle the substantial > incremental load from adding FFmpeg without removing libav. You may > be leery for a while. To change t

Re: [FFmpeg-devel] A patch for gdigrab function

2014-08-16 Thread Michael Niedermayer
On Fri, Aug 08, 2014 at 05:08:46PM +0800, hlszl1...@163.com wrote: > hi, all > > I'm using gdigrab feature on windows, and found that the count of gdi objects > owned by ffmpeg.exe increasing endlessly... > > Here is the patch, can someone commit it? > --

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-16 Thread Russ Allbery
Ivan Kalvachev writes: > I'm quite sure the Security team is full of capable people who can > handle one more package. One, no, this statement is not correct. Not because the security team is not capable -- they are very capable -- but because they are not *full*. You imply that the security te

Re: [FFmpeg-devel] [PATCH 1/2] postproc: use av_freep() for saftey

2014-08-16 Thread Michael Niedermayer
On Sat, Aug 16, 2014 at 10:02:50PM +0200, Clément Bœsch wrote: > On Sat, Aug 16, 2014 at 07:54:24PM +0200, Michael Niedermayer wrote: > > Signed-off-by: Michael Niedermayer > > --- > > libpostproc/postprocess.c | 22 +++--- > > 1 file changed, 11 insertions(+), 11 deletions(-) >

Re: [FFmpeg-devel] [PATCH 1/2] postproc: use av_freep() for saftey

2014-08-16 Thread Clément Bœsch
On Sat, Aug 16, 2014 at 07:54:24PM +0200, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libpostproc/postprocess.c | 22 +++--- > 1 file changed, 11 insertions(+), 11 deletions(-) > > diff --git a/libpostproc/postprocess.c b/libpostproc/postprocess.c >

Re: [FFmpeg-devel] [PATCH] swresample: Add AVFrame based API

2014-08-16 Thread Michael Niedermayer
On Mon, Aug 11, 2014 at 01:49:19AM +0200, Michael Niedermayer wrote: > From: Luca Barbato > > TODO:bump > > Signed-off-by: Michael Niedermayer patch applied with all requested changes [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Republics decline into de

Re: [FFmpeg-devel] Invitation to VDD and registration

2014-08-16 Thread Jean-Baptiste Kempf
On 16 Aug, Michael Niedermayer wrote : > VDD is a great thing for thouse who like meetings and conferences. BS. It's exactly the opposite. It's for people like us, who don't like conferences, but understand the need to meet. > But iam not a conference / meeting / party / traveller guy. I really

Re: [FFmpeg-devel] [PATCH] avformat/metadata: Implement AVFMT_FLAG_NO_META_CONV

2014-08-16 Thread wm4
On Mon, 11 Aug 2014 15:15:45 +0300 Mohammad Alsaleh wrote: > Decoders/parsers run ff_metadata_conv() unconditionally. This makes > it impossible to retrieve or keep original metadata field names. > > If one wishes to skip ff_metadata_conv() calls at the decoding/parsing > stage. This patch i

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-16 Thread Bálint Réczey
Hi Nicolas, 2014-08-16 17:11 GMT+02:00 Nicolas George : > L'octidi 28 thermidor, an CCXXII, Bálint Réczey a écrit : >> Using Gerrit and "file ownersip" are not mutually exclusive. Gerrit >> can be configured to automatically invite the right people for review >> based on the changed path. We recen

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-16 Thread Ivan Kalvachev
On 8/16/14, Pau Garcia i Quiles wrote: > On Sat, Aug 16, 2014 at 5:30 PM, Nicolas George wrote: > > >> The only option is to make sure the users do not suffer from the fork, by >> making sure they can easily use the version that is most suited for their >> need without being sucked into the devel

Re: [FFmpeg-devel] [PATCH] mpegts: always parse pcr

2014-08-16 Thread Michael Niedermayer
On Sat, Aug 16, 2014 at 03:29:16PM +0200, Marton Balint wrote: > > On Sun, 10 Aug 2014, Marton Balint wrote: > > >Previously pcr transmitted without payload but as part of the video stream > >was > >not parsed. > > > >Signed-off-by: Marton Balint > >--- > >libavformat/mpegts.c | 12 +---

Re: [FFmpeg-devel] [PATCH]Fix remote ffprobe fate tests

2014-08-16 Thread Michael Niedermayer
On Sat, Aug 16, 2014 at 04:14:18PM +0200, Paul B Mahol wrote: > On 8/16/14, Carl Eugen Hoyos wrote: > > Hi! > > > > I cannot really test but I hope attached patch fixes the remaining fate > > tests that don't work on remote fate targets. > > > > Please review and push, Carl Eugen > > Missing sign

[FFmpeg-devel] [PATCH 2/2] avformat/movenchint: use av_freep() for saftey

2014-08-16 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavformat/movenchint.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/movenchint.c b/libavformat/movenchint.c index 2602254..006aa09 100644 --- a/libavformat/movenchint.c +++ b/libavformat/movenchint.c @@ -72,7 +72,7

[FFmpeg-devel] [PATCH 1/2] postproc: use av_freep() for saftey

2014-08-16 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libpostproc/postprocess.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/libpostproc/postprocess.c b/libpostproc/postprocess.c index 794cd30..397e4fb 100644 --- a/libpostproc/postprocess.c +++ b/libpostproc/pos

Re: [FFmpeg-devel] Invitation to VDD and registration

2014-08-16 Thread Kieran Kunhya
> And do not forget the language barrier: when writing a mail, there is as > much time as necessary to find words. IRL, the people struggling to find > their words are at a clear disadvantage. We can rotate between different languages every ten minutes =p __

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-16 Thread Pau Garcia i Quiles
On Sat, Aug 16, 2014 at 5:30 PM, Nicolas George wrote: > The only option is to make sure the users do not suffer from the fork, by > making sure they can easily use the version that is most suited for their > need without being sucked into the developers' disagreements. > Can we get back on top

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-16 Thread wm4
On Sat, 16 Aug 2014 23:29:56 +0800 Thomas Goirand wrote: > On 08/15/2014 11:53 PM, The Wanderer wrote: > > It's also something the Linux kernel is still doing, with apparent > > success. > > Yes, the Linux kernel is a successful project. Does this mean using a > list for reviewing patches is a g

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-16 Thread Thomas Goirand
On 08/15/2014 11:53 PM, The Wanderer wrote: > It's also something the Linux kernel is still doing, with apparent > success. Yes, the Linux kernel is a successful project. Does this mean using a list for reviewing patches is a good thing? No! The workflow with a list is simply horrible. Using git-r

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-16 Thread Nicolas George
Le septidi 27 thermidor, an CCXXII, Thomas Goirand a écrit : > If you guys could find a solution to try to work together > again, and merge back both projects, that'd be best for everyone. When people suggest that, I always wonder how they see that happening with regard to the code.

Re: [FFmpeg-devel] Invitation to VDD and registration

2014-08-16 Thread Nicolas George
Le sextidi 26 thermidor, an CCXXII, Stefano Sabatini a écrit : > anybody is planning to go there? I remember that the registration will > close on August 20 (that is, the next week Wednesday), bye. I intend to be there; I have already registered. I have not yet booked a flight as I must wait for m

Re: [FFmpeg-devel] Invitation to VDD and registration

2014-08-16 Thread Nicolas George
Le nonidi 29 thermidor, an CCXXII, Clément Bœsch a écrit : > <+nevcairiel> most people that insist on them are those that know that > they are more experienced and comfortable with them, and hope to win the > argument that way As someone not much at ease with socializing, I can only concur. And d

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-16 Thread Nicolas George
L'octidi 28 thermidor, an CCXXII, Bálint Réczey a écrit : > Using Gerrit and "file ownersip" are not mutually exclusive. Gerrit > can be configured to automatically invite the right people for review > based on the changed path. We recently migrated to Gerrit at the > Wireshark project and it helps

Re: [FFmpeg-devel] Invitation to VDD and registration

2014-08-16 Thread Michael Niedermayer
On Sat, Aug 16, 2014 at 04:29:13PM +0200, wm4 wrote: > On Sat, 16 Aug 2014 15:12:55 +0100 > Kieran Kunhya wrote: > > > > like i said, iam happy to resign, if that reunites the projects and > > > resolves the problems > > > how can my resignation not be enough, if as you say iam the blocking > > >

Re: [FFmpeg-devel] Invitation to VDD and registration

2014-08-16 Thread Kieran Kunhya
> The social aspect > should IMO not overlap over the technical one. Problems with the way people behave are social problems... In an ideal world yes, all the disagreements could be technical but in reality they are sadly not all technical. Kieran ___ f

Re: [FFmpeg-devel] Invitation to VDD and registration

2014-08-16 Thread wm4
On Sat, 16 Aug 2014 15:12:55 +0100 Kieran Kunhya wrote: > > like i said, iam happy to resign, if that reunites the projects and > > resolves the problems > > how can my resignation not be enough, if as you say iam the blocking > > factor ? > > You impose conditions on your resignation. We proba

Re: [FFmpeg-devel] Invitation to VDD and registration

2014-08-16 Thread Clément Bœsch
On Sat, Aug 16, 2014 at 02:36:00PM +0100, Kieran Kunhya wrote: [...] > Whether you like it or not these problems are social issues (cf VDD11) > and they will require a social solution. You are the blocking factor > and everyone knows it and we say it every VDD. I'm too lazy to make a long mail to

Re: [FFmpeg-devel] [PATCH]Fix remote ffprobe fate tests

2014-08-16 Thread Paul B Mahol
On 8/16/14, Carl Eugen Hoyos wrote: > Hi! > > I cannot really test but I hope attached patch fixes the remaining fate > tests that don't work on remote fate targets. > > Please review and push, Carl Eugen Missing signed-off. ___ ffmpeg-devel mailing lis

Re: [FFmpeg-devel] Invitation to VDD and registration

2014-08-16 Thread Kieran Kunhya
> like i said, iam happy to resign, if that reunites the projects and > resolves the problems > how can my resignation not be enough, if as you say iam the blocking > factor ? You impose conditions on your resignation. And wow, you'd rather resign than meet in person... __

Re: [FFmpeg-devel] [PATCH 4/4] dpx: fix endianess for RGB 8bits

2014-08-16 Thread Michael Niedermayer
On Sat, Aug 16, 2014 at 01:57:16PM +0200, Christophe Gisquet wrote: > Hi, > > 2014-08-16 12:51 GMT+02:00 Michael Niedermayer : > > On Sat, Aug 16, 2014 at 11:39:44AM +0200, Christophe Gisquet wrote: > [...] > > i added hoger to the CC > > Yes, I read that file. I sent minutes after my previous ma

Re: [FFmpeg-devel] Invitation to VDD and registration

2014-08-16 Thread Michael Niedermayer
On Sat, Aug 16, 2014 at 02:36:00PM +0100, Kieran Kunhya wrote: > > do you think me, the rest of the FFmpeg and Libav developers sitting > > together and people accusing each other of theft and lies. Would lead > > to improved relations ? > > You perhaps fail to realise since you've never done it b

Re: [FFmpeg-devel] [PATCH 6/7] avcodec/hevc_mvs: dont redundantly initialize ref_idx_curr

2014-08-16 Thread Michael Niedermayer
On Sat, Aug 16, 2014 at 03:02:08PM +0200, James Darnley wrote: > On 2014-08-16 01:15, Michael Niedermayer wrote: > > Signed-off-by: Michael Niedermayer > > --- > > libavcodec/hevc_mvs.c |2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > fine applied thx [...] -- Michael

Re: [FFmpeg-devel] [PATCH 7/7] avcodec/mips/compute_antialias_float: remove unused variable

2014-08-16 Thread Michael Niedermayer
On Sat, Aug 16, 2014 at 03:04:04PM +0200, James Darnley wrote: > On 2014-08-16 01:15, Michael Niedermayer wrote: > > Signed-off-by: Michael Niedermayer > > --- > > libavcodec/mips/compute_antialias_float.h |3 --- > > 1 file changed, 3 deletions(-) > > It is unused so that is fine. applied

Re: [FFmpeg-devel] [PATCH 3/7] avcodec/dvbsubdec: dont mix integers with pointers

2014-08-16 Thread Michael Niedermayer
On Sat, Aug 16, 2014 at 02:55:44PM +0200, James Darnley wrote: > On 2014-08-16 01:15, Michael Niedermayer wrote: > > Signed-off-by: Michael Niedermayer > > --- > > libavcodec/dvbsubdec.c |4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > Fine, both pointers. applied thx [...

Re: [FFmpeg-devel] [PATCH 5/7] avformat/udp: dont mix integers with pointers

2014-08-16 Thread Michael Niedermayer
On Sat, Aug 16, 2014 at 03:00:32PM +0200, James Darnley wrote: > On 2014-08-16 01:15, Michael Niedermayer wrote: > > Signed-off-by: Michael Niedermayer > > --- > > libavformat/udp.c |4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/libavformat/udp.c b/libavfor

Re: [FFmpeg-devel] [PATCH 2/7] avcodec/dvdsubdec: Dont mix integers with pointers

2014-08-16 Thread Michael Niedermayer
On Sat, Aug 16, 2014 at 02:53:45PM +0200, James Darnley wrote: > On 2014-08-16 01:15, Michael Niedermayer wrote: > > Signed-off-by: Michael Niedermayer > > --- > > libavcodec/dvdsubdec.c |4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > Fine, clearly a pointer. applied thx

Re: [FFmpeg-devel] Invitation to VDD and registration

2014-08-16 Thread Kieran Kunhya
> do you think me, the rest of the FFmpeg and Libav developers sitting > together and people accusing each other of theft and lies. Would lead > to improved relations ? You perhaps fail to realise since you've never done it before that there's vastly less of these long, stupid historical tales whe

Re: [FFmpeg-devel] [PATCH] mpegts: always parse pcr

2014-08-16 Thread Marton Balint
On Sun, 10 Aug 2014, Marton Balint wrote: Previously pcr transmitted without payload but as part of the video stream was not parsed. Signed-off-by: Marton Balint --- libavformat/mpegts.c | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) Hello Michael, Please merge from my

Re: [FFmpeg-devel] [PATCH 2/3] build: rely on pkg-config for libx264 probing

2014-08-16 Thread Clément Bœsch
On Fri, Aug 15, 2014 at 06:49:39AM +, Carl Eugen Hoyos wrote: [...] > > But really that's insane, because I know you will > > end-up hardcoding all kind of linker flags to > > these fallback calls, > > Yes, some external libraries that never have > additional dependencies currently depend o

Re: [FFmpeg-devel] [PATCH 4/4] dpx: fix endianess for RGB 8bits

2014-08-16 Thread Jean First
On Sat Aug 16 2014 13:57:16 GMT+0200 (CEST), Christophe Gisquet wrote: Hi, 2014-08-16 12:51 GMT+02:00 Michael Niedermayer : On Sat, Aug 16, 2014 at 11:39:44AM +0200, Christophe Gisquet wrote: [...] i added hoger to the CC Yes, I read that file. I sent minutes after my previous mail inquiries

Re: [FFmpeg-devel] [PATCH 7/7] avcodec/mips/compute_antialias_float: remove unused variable

2014-08-16 Thread James Darnley
On 2014-08-16 01:15, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavcodec/mips/compute_antialias_float.h |3 --- > 1 file changed, 3 deletions(-) It is unused so that is fine. signature.asc Description: OpenPGP digital signature

Re: [FFmpeg-devel] [PATCH 6/7] avcodec/hevc_mvs: dont redundantly initialize ref_idx_curr

2014-08-16 Thread James Darnley
On 2014-08-16 01:15, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavcodec/hevc_mvs.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > fine signature.asc Description: OpenPGP digital signature ___ ffmpeg-de

Re: [FFmpeg-devel] [PATCH 5/7] avformat/udp: dont mix integers with pointers

2014-08-16 Thread James Darnley
On 2014-08-16 01:15, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavformat/udp.c |4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavformat/udp.c b/libavformat/udp.c > index a8baa5b..8412eeb 100644 > --- a/libavformat/udp.c > +++ b/l

Re: [FFmpeg-devel] [PATCH 3/7] avcodec/dvbsubdec: dont mix integers with pointers

2014-08-16 Thread James Darnley
On 2014-08-16 01:15, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavcodec/dvbsubdec.c |4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Fine, both pointers. signature.asc Description: OpenPGP digital signature ___

Re: [FFmpeg-devel] [PATCH 2/7] avcodec/dvdsubdec: Dont mix integers with pointers

2014-08-16 Thread James Darnley
On 2014-08-16 01:15, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavcodec/dvdsubdec.c |4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Fine, clearly a pointer. signature.asc Description: OpenPGP digital signature __

Re: [FFmpeg-devel] [PATCH 4/4] dpx: fix endianess for RGB 8bits

2014-08-16 Thread Christophe Gisquet
Hi, 2014-08-16 12:51 GMT+02:00 Michael Niedermayer : > On Sat, Aug 16, 2014 at 11:39:44AM +0200, Christophe Gisquet wrote: [...] > i added hoger to the CC Yes, I read that file. I sent minutes after my previous mail inquiries to various DPX users/developers I've seen through the files/mails/bugre

Re: [FFmpeg-devel] Invitation to VDD and registration

2014-08-16 Thread compn
On Sat, 16 Aug 2014 05:16:09 +0200 Michael Niedermayer wrote: > I do want to resolve this, and i think if for example libav developers > and ffmpeg developers would each say "shit happened, we all made > mistakes" thats a start from which i think projects could be reunited. diego said those exac

Re: [FFmpeg-devel] [PATCH 3/4] dict.c: Free non-strduped av_dict_set arguments on error.

2014-08-16 Thread Reimar Döffinger
On Mon, Aug 11, 2014 at 09:17:18PM +0200, Reimar Döffinger wrote: > Unfortunately this was not explicitly documented and thus > might be very risky. > But basically all uses I saw in FFmpeg had a memleak in these > cases. All others pushed, this one still needs some discussion. ___

Re: [FFmpeg-devel] [PATCH 4/4] dpx: fix endianess for RGB 8bits

2014-08-16 Thread Michael Niedermayer
On Sat, Aug 16, 2014 at 11:39:44AM +0200, Christophe Gisquet wrote: > Hi, > > 2014-08-14 11:48 GMT+02:00 Christophe Gisquet : > > Hi, > > > > 2014-08-14 5:01 GMT+02:00 Michael Niedermayer : > >> On Wed, Aug 13, 2014 at 10:21:54AM +, Christophe Gisquet wrote: > >>> case 50081: > >>> +

Re: [FFmpeg-devel] [PATCH 3/4] dict.c: Free non-strduped av_dict_set arguments on error.

2014-08-16 Thread Reimar Döffinger
On Wed, Aug 13, 2014 at 01:57:56PM +0200, Michael Niedermayer wrote: > On Mon, Aug 11, 2014 at 09:17:18PM +0200, Reimar Döffinger wrote: > > Unfortunately this was not explicitly documented and thus > > might be very risky. > > yes > > maybe a AV_DICT_FREE_ARGS_ON_ERROR could be used, but its not

Re: [FFmpeg-devel] [PATCH 3/4] dict.c: Free non-strduped av_dict_set arguments on error.

2014-08-16 Thread Reimar Döffinger
On Mon, Aug 11, 2014 at 10:11:59PM +0200, wm4 wrote: > On Mon, 11 Aug 2014 21:17:18 +0200 > Reimar Döffinger wrote: > > > Unfortunately this was not explicitly documented and thus > > might be very risky. > > But basically all uses I saw in FFmpeg had a memleak in these > > cases. > > It's the m

[FFmpeg-devel] [PATCH]Fix remote ffprobe fate tests

2014-08-16 Thread Carl Eugen Hoyos
Hi! I cannot really test but I hope attached patch fixes the remaining fate tests that don't work on remote fate targets. Please review and push, Carl EugenFrom 5c2e7283bb1db659246da10d6f1f078123497207 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Sat, 16 Aug 2014 12:19:05 +0200 Subje

Re: [FFmpeg-devel] [PATCH 4/7] avformat/thp: dont mix integers with pointers

2014-08-16 Thread Michael Niedermayer
On Sat, Aug 16, 2014 at 10:21:57AM +0200, Paul B Mahol wrote: > On 8/16/14, Michael Niedermayer wrote: > > Signed-off-by: Michael Niedermayer > > --- > > libavformat/thp.c |2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/libavformat/thp.c b/libavformat/thp.c > >

Re: [FFmpeg-devel] Invitation to VDD and registration

2014-08-16 Thread Michael Niedermayer
On Sat, Aug 16, 2014 at 09:48:18AM +0200, Jean-Baptiste Kempf wrote: > On 16 Aug, Michael Niedermayer wrote : > > On Sat, Aug 16, 2014 at 03:48:53AM +0200, Jean-Baptiste Kempf wrote: > > > On 16 Aug, Michael Niedermayer wrote : > > > > I had not intended to come to VDD and reading this > > > > http

Re: [FFmpeg-devel] [PATCH 4/4] dpx: fix endianess for RGB 8bits

2014-08-16 Thread Christophe Gisquet
Hi, 2014-08-14 11:48 GMT+02:00 Christophe Gisquet : > Hi, > > 2014-08-14 5:01 GMT+02:00 Michael Niedermayer : >> On Wed, Aug 13, 2014 at 10:21:54AM +, Christophe Gisquet wrote: >>> case 50081: >>> +avctx->pix_fmt = AV_PIX_FMT_BGR24; >>> +break; >> >> this possibly breaks d

Re: [FFmpeg-devel] Invitation to VDD and registration

2014-08-16 Thread Reimar Döffinger
On 16.08.2014, at 09:50, Jean-Baptiste Kempf wrote: > On 16 Aug, Reimar Döffinger wrote : >> Which is mostly why people avoid speaking to each other, at the least about >> anything close to the subject, so far the best that would happen is that >> everyone felt uncomfortable. > > VDD is not abo

[FFmpeg-devel] AECHO

2014-08-16 Thread Yamusani Vinay
Hi All, I included aecho(aecho.c) filter in ffmpeg 2.3.4,I compiled the library and so files are generated I included those so files in my android application I added that filter and ran my application but the song is not playing can anyone help me in solving this problem Thanks, Vinay Yamusani.

Re: [FFmpeg-devel] [PATCH 4/7] avformat/thp: dont mix integers with pointers

2014-08-16 Thread Paul B Mahol
On 8/16/14, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavformat/thp.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/thp.c b/libavformat/thp.c > index bc4f0daf..714cec6 100644 > --- a/libavformat/thp.c > +++ b/libavformat/t

Re: [FFmpeg-devel] Small patch for someone to push please

2014-08-16 Thread JULIAN GARDNER
> > From: Michael Niedermayer >To: FFmpeg development discussions and patches >Sent: Saturday, 16 August 2014, 1:42 >Subject: Re: [FFmpeg-devel] Small patch for someone to push please > > >On Fri, Aug 15, 2014 at 11:26:40AM +0100, JULIAN GARDNER wrote: >>

Re: [FFmpeg-devel] Invitation to VDD and registration

2014-08-16 Thread Jean-Baptiste Kempf
On 16 Aug, Reimar Döffinger wrote : > Which is mostly why people avoid speaking to each other, at the least about > anything close to the subject, so far the best that would happen is that > everyone felt uncomfortable. VDD is not about your projects fighting each other. While you were fighting,

Re: [FFmpeg-devel] Invitation to VDD and registration

2014-08-16 Thread Jean-Baptiste Kempf
On 16 Aug, Michael Niedermayer wrote : > On Sat, Aug 16, 2014 at 03:48:53AM +0200, Jean-Baptiste Kempf wrote: > > On 16 Aug, Michael Niedermayer wrote : > > > I had not intended to come to VDD and reading this > > > https://lists.debian.org/debian-devel/2014/08/msg00478.html > > > > > > I definiti

Re: [FFmpeg-devel] Invitation to VDD and registration

2014-08-16 Thread Reimar Döffinger
On 16.08.2014, at 03:48, Jean-Baptiste Kempf wrote: > On 16 Aug, Michael Niedermayer wrote : >> I had not intended to come to VDD and reading this >> https://lists.debian.org/debian-devel/2014/08/msg00478.html >> >> I definitively will not come to VDD, as it would just be wasted time, > > That's