Merge lp:~vikoadi/audience/fix-nonvideo into lp:~audience-members/audience/trunk

Proposed by Viko Adi Rahmawan
Status: Merged
Approved by: Cody Garver
Approved revision: 374
Merged at revision: 387
Proposed branch: lp:~vikoadi/audience/fix-nonvideo
Merge into: lp:~audience-members/audience/trunk
Diff against target: 25 lines (+2/-2)
2 files modified
src/Widgets/PreviewPopover.vala (+1/-1)
src/Widgets/VideoPlayer.vala (+1/-1)
To merge this branch: bzr merge lp:~vikoadi/audience/fix-nonvideo
Reviewer Review Type Date Requested Status
Audience Members Pending
Review via email: mp+231987@code.launchpad.net

Commit message

Don't try to get video size if doesn't have a video stream

Description of the change

dont try to get video size if doesnt have videostream

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/Widgets/PreviewPopover.vala'
--- src/Widgets/PreviewPopover.vala 2014-06-30 14:54:30 +0000
+++ src/Widgets/PreviewPopover.vala 2014-08-23 10:37:11 +0000
@@ -68,7 +68,7 @@
68 try {68 try {
69 var info = new Gst.PbUtils.Discoverer (10 * Gst.SECOND).discover_uri (uri);69 var info = new Gst.PbUtils.Discoverer (10 * Gst.SECOND).discover_uri (uri);
70 var video = info.get_video_streams ();70 var video = info.get_video_streams ();
71 if (video.data != null) {71 if (video != null && video.data != null) {
72 var video_info = (Gst.PbUtils.DiscovererVideoInfo)video.data;72 var video_info = (Gst.PbUtils.DiscovererVideoInfo)video.data;
73 uint video_width = video_info.get_width ();73 uint video_width = video_info.get_width ();
74 uint video_height = video_info.get_height ();74 uint video_height = video_info.get_height ();
7575
=== modified file 'src/Widgets/VideoPlayer.vala'
--- src/Widgets/VideoPlayer.vala 2014-06-30 14:54:30 +0000
+++ src/Widgets/VideoPlayer.vala 2014-08-23 10:37:11 +0000
@@ -99,7 +99,7 @@
99 try {99 try {
100 var info = new Gst.PbUtils.Discoverer (10 * Gst.SECOND).discover_uri (value);100 var info = new Gst.PbUtils.Discoverer (10 * Gst.SECOND).discover_uri (value);
101 var video = info.get_video_streams ();101 var video = info.get_video_streams ();
102 if (video.data != null) {102 if (video != null && video.data != null) {
103 var video_info = (Gst.PbUtils.DiscovererVideoInfo)video.data;103 var video_info = (Gst.PbUtils.DiscovererVideoInfo)video.data;
104 video_width = video_info.get_width ();104 video_width = video_info.get_width ();
105 video_height = video_info.get_height ();105 video_height = video_info.get_height ();

Subscribers

People subscribed via source and target branches