Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

Hello, Happy New Year. Any news about this patch? Maybe Carl Eugen Hoyos could help us with this? Best regards. On Thu, Dec 6, 2018 at 6:10 PM Vasile Toncu wrote: > Hi, > > Since this patch is passing all the current tests, all the modification > required by Thomas were implemented and from

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

Hi, Since this patch is passing all the current tests, all the modification required by Thomas were implemented and from our point of view it satisfies all requirements for adding the double license to the filter, please approve this patch for integration. Also, there are two more patches blocked

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

Hello, Thank you Thomas for the reviews and support! The proposed actions, by Nicolas George in https://ffmpeg.org/pipermail/ffmpeg-devel/2017-December/223072.html, were taken. >* Can one simply change the tinterlace from GPL to LGPL? * Of course not. There are two non-simple courses of action

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

Hi, 2018-08-16 13:56 GMT+02:00 Vasile Toncu : > Hi, > > Thank you for the additional testing effort. > Fixed the issue. > > thanks, the patch looks good to me as far as I can judge. It´s up to more experienced developers now to permit the license change. Can anybody please have a look at this.

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

Hi, Thank you for the additional testing effort. Fixed the issue. On Tue, Aug 14, 2018 at 10:25 PM, Thomas Mundt wrote: > Hi, > > 2018-08-14 18:53 GMT+02:00 Vasile Toncu : > > > Hi Thomas, > > > > I added the log messages. > > > > Thanks for the review. > > > > I found some more time for

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

Hi, 2018-08-14 18:53 GMT+02:00 Vasile Toncu : > Hi Thomas, > > I added the log messages. > > Thanks for the review. > I found some more time for testing this evening. Unfortunately there are still issues. I get half green pictures with some modes at high bit depth. But the fix is easy. The

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

Hi Thomas, I added the log messages. Thanks for the review. On Tue, Aug 14, 2018 at 3:23 PM, Thomas Mundt wrote: > Hi, > > 2018-08-13 0:02 GMT+02:00 Vasile Toncu : > > > Hello, > > > > I have updated patch 3 according to review, removed all code related to > the > > new flags (MERGE_TFF,

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

Hi, 2018-08-13 0:02 GMT+02:00 Vasile Toncu : > Hello, > > I have updated patch 3 according to review, removed all code related to the > new flags (MERGE_TFF, MERGE_BFF) and threading functionality. > > Thanks, your patch looks fine to me now. There is just one thing, I forgot to mention last

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

Hello, I have updated patch 3 according to review, removed all code related to the new flags (MERGE_TFF, MERGE_BFF) and threading functionality. On Sat, Jul 28, 2018 at 5:04 PM, Thomas Mundt wrote: > Hi, > > 2018-07-24 12:17 GMT+02:00 Vasile Toncu : > > > Fixed tabs. > > Thank you for the

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

Hi, 2018-07-24 12:17 GMT+02:00 Vasile Toncu : > Fixed tabs. > Thank you for the feedback. > > {} > +case MODE_INTERLEAVE_BOTTOM: > +case MODE_INTERLEAVE_TOP: > +y = y * 2; > + > +if (tinterlace->flags & FLAG_VLPF || tinterlace->flags & > FLAG_CVLPF) { > + > +

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

2018-07-25 17:11 GMT+02:00 Vasile Toncu : > What would be the next steps? > > On Tue, Jul 24, 2018 at 12:17 PM, Vasile Toncu > wrote: > > > Fixed tabs. > > Thank you for the feedback. > > > > On Fri, Jul 20, 2018 at 7:27 PM, Paul B Mahol wrote: > > > >> On 7/20/18, Vasile Toncu wrote: > >> >

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

On 7/25/18, Vasile Toncu wrote: > What would be the next steps? > Ping patch every second. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

What would be the next steps? On Tue, Jul 24, 2018 at 12:17 PM, Vasile Toncu wrote: > Fixed tabs. > Thank you for the feedback. > > On Fri, Jul 20, 2018 at 7:27 PM, Paul B Mahol wrote: > >> On 7/20/18, Vasile Toncu wrote: >> > Hi, >> > >> > Did you have the time to take a look at patches 2

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

Fixed tabs. Thank you for the feedback. On Fri, Jul 20, 2018 at 7:27 PM, Paul B Mahol wrote: > On 7/20/18, Vasile Toncu wrote: > > Hi, > > > > Did you have the time to take a look at patches 2 and 3? It's everything > > fine? > > Yes and no, patches have tabs. Please fix that. >

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

On 7/20/18, Vasile Toncu wrote: > Hi, > > Did you have the time to take a look at patches 2 and 3? It's everything > fine? Yes and no, patches have tabs. Please fix that. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

Hi, Did you have the time to take a look at patches 2 and 3? It's everything fine? Regards, Vasile On Mon, Jul 16, 2018 at 5:51 PM, Vasile Toncu wrote: > Hello, > > I have updated patch 3, it shall work now. > > Thank you, > Vasile Toncu > ___

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

Hello, I have updated patch 3, it shall work now. Thank you, Vasile Toncu From 23bdcea190ccb801e3d5a0eb081fb28b17144452 Mon Sep 17 00:00:00 2001 From: Vasile Toncu Date: Mon, 16 Jul 2018 16:13:10 +0200 Subject: [PATCH] Patch 3 - Move reinterlace to tinterlace --- configure

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

Hi, 2018-05-30 16:10 GMT+02:00 Vasile Toncu : > Hello, > > I've sent a wrong version in the previous email for patch 3. Please > ignore. This is the corect one. the compiler warnings are gone, but fate-filter-pixfmts-tinterlace_pad still fails. Why do you replace the usage of draw utils for

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

Hello, I've sent a wrong version in the previous email for patch 3. Please ignore. This is the corect one. Have a good day, Vasile Toncu >From 5898c531066f8e6c034eb1c882000f60ba4f0cb9 Mon Sep 17 00:00:00 2001 From: Vasile Toncu Date: Tue, 29 May 2018 18:29:06 +0300 Subject: [PATCH]

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

Hello, Here is patch 3 which renames reinterlace to tinterlace. Fate tests and warnings should be ok now. In patch 4 I will add the new options to the filter. Thank you, Vasile Toncu >From 25d4f0154fe2a8bde3a47491f80ce131ebb20330 Mon Sep 17 00:00:00 2001 From: Vasile Toncu Date: Tue, 29 May

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

Hi, 2018-04-25 16:24 GMT+02:00 Vasile Toncu : > Hello, > > Here is patch 3. I renamed reinterlace to tinterlace and replaced the > functionality of tinterlace with the one from reinterlace. Please review. > > Thank you, > Vasile > Same as I wrote for your previous

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

On 4/25/2018 7:54 PM, Vasile Toncu wrote: Hello, Here is patch 3. I renamed reinterlace to tinterlace and replaced the functionality of tinterlace with the one from reinterlace. Please review. If any options are new or different from the original tinterlace filter, please add doc entries

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

Hello, Here is patch 3. I renamed reinterlace to tinterlace and replaced the functionality of tinterlace with the one from reinterlace. Please review. Thank you, Vasile >From ee33ed769207a99adb193eece32a5043ff0b6d94 Mon Sep 17 00:00:00 2001 From: Vasile Toncu Date: