Hi,<br>

<br>

First off, I'm not on the dev list, so please CC me on any replies.<br>

<br>

MythDVD seems to incorrectly identify widescreen/letter boxed DVDs.
There seem to be 4 types of DVDs that can be detected, 16:9 or 4:3 and
letterboxed or not for each. I don't really understand what 16:19
non-letterboxed is or what 4:3 letterboxed would be, but that's not
necessarily the base of this problem.<br>

<br>

The big problem here is that MythDVD checks the letterboxed variable on
the video_attributes struct to see if the DVD is LB or not, but
libdvdread no longer/never did actually set a value other than a 0 in
that variable. A quick grep through the libdvdread sources shows that
it is not used. The following appears to fix things on my end. It also
seems that the 16:9 vs. 4:3 check should be sufficient, but if it's not
then the following should definitely fix it anyway:<br>

<br>

Index: mtd/dvdprobe.cpp<br>

==============================
<div id="mb_0">=====================================<br>
--- mtd/dvdprobe.cpp&nbsp;&nbsp;&nbsp; (revision 8587)<br>
+++ mtd/dvdprobe.cpp&nbsp;&nbsp;&nbsp; (working copy)<br>
@@ -703,7 +703,7 @@<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
cerr &lt;&lt; &quot;dvdprobe.o: Could not get video format for a title&quot;
&lt;&lt; endl;<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
}<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
<br>
-&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
if(video_attributes-&gt;letterboxed)<br>
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
if(video_attributes-&gt;permitted_df == 2)<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
{<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
new_title-&gt;setLBox(true);<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
}<br>
<br>
<br>
I'd appreciate it if anyone who knows what 16:9 non-LB or 4:3 LB is would let me know.<br>
<br>
Thanks,<br>
Matt Schulkind</div>