[pitivi] mainwindow: Actually use DAR (not PAR) to set the viewer's display aspect ratio



commit 6ca7dd06dc54d02969fc2722f720f0d5dd77ea36
Author: Jean-François Fortin Tam <nekohayo gmail com>
Date:   Fri Jan 10 21:15:16 2014 -0500

    mainwindow: Actually use DAR (not PAR) to set the viewer's display aspect ratio
    
    Using the pixel aspect ratio to set the viewer's (display) aspect ratio
    made no sense. We had a nice unused method for that exact purpose.
    
    This also happens to fix bug #722003

 pitivi/mainwindow.py |    2 +-
 pitivi/project.py    |    6 ------
 2 files changed, 1 insertions(+), 7 deletions(-)
---
diff --git a/pitivi/mainwindow.py b/pitivi/mainwindow.py
index 57c8331..0591b03 100644
--- a/pitivi/mainwindow.py
+++ b/pitivi/mainwindow.py
@@ -1047,7 +1047,7 @@ class PitiviMainWindow(Gtk.Window, Loggable):
         """
         When the project setting change, we reset the viewer aspect ratio
         """
-        self.viewer.setDisplayAspectRatio(project.aspect_ratio)
+        self.viewer.setDisplayAspectRatio(project.getDAR())
         self.viewer.timecode_entry.setFramerate(project.videorate)
 
     def _sourceListMissingPluginsCb(self, unused_project, unused_uri, unused_factory,
diff --git a/pitivi/project.py b/pitivi/project.py
index 253568c..9bdd751 100644
--- a/pitivi/project.py
+++ b/pitivi/project.py
@@ -712,12 +712,6 @@ class Project(Loggable, GES.Project):
             self._emitChange("rendering-settings-changed", "pixel-aspect-ratio", value)
 
     @property
-    def aspect_ratio(self):
-        videopar = self.videopar
-        par = videopar.num / videopar.denom
-        return par * float(self.videowidth) / float(self.videoheight)
-
-    @property
     def audiochannels(self):
         return self.audio_profile.get_restriction()[0]["channels"]
 


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]