From: Ben Hutchings <ben@decadent.org.uk>
Date: Sat, 7 Jul 2007 18:26:28 +0000 (+0000)
Subject: Added vertical padding of buttons to even y coordinates since dvdauthor claims odd... 
X-Git-Tag: 1.2~1
X-Git-Url: https://git.decadent.org.uk/gitweb/?p=videolink.git;a=commitdiff_plain;h=7c0ec4cf5e77d68a7e2313c40e2038ff0cdcbe69

Added vertical padding of buttons to even y coordinates since dvdauthor claims odd values may result in incorrect display.
---

diff --git a/generate_dvd.cpp b/generate_dvd.cpp
index 45e9ca6..1d43a9a 100644
--- a/generate_dvd.cpp
+++ b/generate_dvd.cpp
@@ -211,11 +211,15 @@ void dvd_generator::generate_menu_vob(unsigned index,
 		}
 	    }
 	}
+	// Pad vertically to even y coordinates since dvdauthor claims
+	// odd values may result in incorrect display.
+	// XXX This may cause overlappping where it wasn't previously
+	// a problem.
 	spumux_file << "      <button"
 	    " x0='" << this_entry.area.left << "'"
-	    " y0='" << this_entry.area.top << "'"
+	    " y0='" << (this_entry.area.top & ~1) << "'"
 	    " x1='" << this_entry.area.right << "'"
-	    " y1='" << this_entry.area.bottom << "'"
+	    " y1='" << ((this_entry.area.bottom + 1) & ~1) << "'"
 	    " left='" << (i == 0 ? button_count : i) << "'"
 	    " right='" << 1 + (i + 1) % button_count << "'"
 	    " up='" << 1 + up_button << "'"