[mythtv] Ticket #4872: Detect if video is letterboxed and switch to corect fill mode

[Mårten Bohlin]

2008/3/6 Mark Buechler <mark.buechler at gmail.com>:
> Hi
>
>
>
> On Wed, Mar 5, 2008 at 7:41 PM, MythTV <mythtv at cvs.mythtv.org> wrote:
> >
> > #4872: Detect if video is letterboxed and switch to corect fill mode
> >
> -------------------------------------+--------------------------------------
> >  Reporter:  Mårten <mabo at mochc.net>  |        Owner:  ijr
> >     Type:  enhancement              |       Status:  new
> >  Priority:  minor                    |    Milestone:  unknown
> >
> > Component:  mythtv                   |      Version:  head
> >  Severity:  medium                   |   Resolution:
> >  Mlocked:  0                        |
> >
> -------------------------------------+--------------------------------------
> >
> > Comment(by davilla at 4pi.com):
> >
> >  How about also moving the boolean test for isDetectLetterbox outside the
> >  routine and use it to trap the call to DetectLetterbox so that 15 lines
> of
> >  initialization code and the call/return does not execute if this feature
> >  is not enabled.
> >
> > --
> > Ticket URL: <http://svn.mythtv.org/trac/ticket/4872#comment:3>
> >
> >
> >
> > MythTV <http://svn.mythtv.org/trac>
> > MythTV
> >
>
> After some more testing, it seems the last patch also has this issue. When
> this happens there aren't any black border at all but for some reason that's
> not detected and it stays in letterbox.
>
> - Mark.
>

Mark could you try out the latest version of the patch (version 6).
This version should be much better at switching back to 4:3.

Now it is enough if there is content in either the top or bottom part.
And earlier it did not switch if there was a solid color that was not
black (as long it was not to bright). Now it also requires that the
bars are black.

Mårten