Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: MythTV: Dev

Re: [mythtv-commits] mythtv commit: r15726 - in trunk/mythtv/filters by danielk

 

 

MythTV dev RSS feed   Index | Next | Previous | View Threaded


msc at antzsystem

Feb 2, 2008, 4:30 AM

Post #1 of 1 (354 views)
Permalink
Re: [mythtv-commits] mythtv commit: r15726 - in trunk/mythtv/filters by danielk

Am Freitag, 1. Februar 2008 schrieb mythtv [at] cvs:
> Author: danielk
> Date: 2008-02-01 20:55:20 +0000 (Fri, 01 Feb 2008)
> New Revision: 15726
> Changeset: http://cvs.mythtv.org/trac/changeset/15726
>
> Modified:
>
> trunk/mythtv/filters/greedyhdeint/filter_greedyhdeint.c
> trunk/mythtv/filters/yadif/filter_yadif.c
>
> Log:
>
> Fixes #4513. A couple small tweaks to greedy/yadiff deinterlacers
> from Mark Kendall.

This fix was wrong. Now there will all material be deinterlaced, neither
you select progressive nor interlace. Cause for preserve frame history
the function:
void NuppelVideoPlayer::SetScanType(FrameScanType scan)
enable deinterlacing for all "double_process" deinterlacers.

We need an additional flag inside deinterlacer plugins for a manually
override from the user or we need to set the interlace-flag inside
frame* structure if the user wants to override.

Two things must be preserved:
- double_process deints should get all frames to preserve history
- deinterlacer should know users decision regarding deinterlacing or
not, if no decision was made he trust the frame-flags

I don't know the impact of changing frame-structure, but i would prefer
this solution.
other opinions?

--
Markus Schulz
_______________________________________________
mythtv-dev mailing list
mythtv-dev [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-dev

MythTV dev RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.