Merge lp:~rsalveti/media-hub/fixing_audio_sink_gst_property_name into lp:media-hub

Proposed by Ricardo Salveti
Status: Merged
Approved by: Thomas Voß
Approved revision: 59
Merged at revision: 60
Proposed branch: lp:~rsalveti/media-hub/fixing_audio_sink_gst_property_name
Merge into: lp:media-hub
Diff against target: 12 lines (+1/-1)
1 file modified
src/core/media/gstreamer/playbin.h (+1/-1)
To merge this branch: bzr merge lp:~rsalveti/media-hub/fixing_audio_sink_gst_property_name
Reviewer Review Type Date Requested Status
Thomas Voß (community) Approve
Review via email: mp+233727@code.launchpad.net

Commit message

playbin: fixing audio-sink gst property name

Description of the change

playbin: fixing audio-sink gst property name

To post a comment you must log in.
Revision history for this message
Thomas Voß (thomas-voss) wrote :

LGTM.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/core/media/gstreamer/playbin.h'
2--- src/core/media/gstreamer/playbin.h 2014-08-29 18:06:11 +0000
3+++ src/core/media/gstreamer/playbin.h 2014-09-08 14:21:28 +0000
4@@ -269,7 +269,7 @@
5 void set_audio_stream_role(media::Player::AudioStreamRole new_audio_role)
6 {
7 GstElement *audio_sink = NULL;
8- g_object_get (pipeline, "audio_sink", &audio_sink, NULL);
9+ g_object_get (pipeline, "audio-sink", &audio_sink, NULL);
10
11 std::string role_str("props,media.role=" + get_audio_role_str(new_audio_role));
12 std::cout << "Audio stream role: " << role_str << std::endl;

Subscribers

People subscribed via source and target branches