Merge lp:~donadigo/wingpanel-indicator-sound/artists-strv-unowned into lp:~wingpanel-devs/wingpanel-indicator-sound/trunk

Proposed by Adam Bieńkowski
Status: Merged
Approved by: Djax
Approved revision: 129
Merged at revision: 129
Proposed branch: lp:~donadigo/wingpanel-indicator-sound/artists-strv-unowned
Merge into: lp:~wingpanel-devs/wingpanel-indicator-sound/trunk
Diff against target: 26 lines (+2/-3)
2 files modified
src/CMakeLists.txt (+1/-1)
src/Widgets/MprisGui.vala (+1/-2)
To merge this branch: bzr merge lp:~donadigo/wingpanel-indicator-sound/artists-strv-unowned
Reviewer Review Type Date Requested Status
WingPanel Devs Pending
Review via email: mp+310193@code.launchpad.net

Commit message

* Make artists arrary unowned

Description of the change

This branch applies the patch from ricotz:
https://paste.debian.net/plain/893308

To post a comment you must log in.
129. By Rico Tzschichholz

Make artists array unowned and bump vala version to 0.26

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/CMakeLists.txt'
2--- src/CMakeLists.txt 2015-06-25 22:03:56 +0000
3+++ src/CMakeLists.txt 2016-11-07 15:14:30 +0000
4@@ -8,7 +8,7 @@
5
6 find_package (Vala REQUIRED)
7 include (ValaVersion)
8-ensure_vala_version ("0.22.0" MINIMUM)
9+ensure_vala_version ("0.26.0" MINIMUM)
10
11 include (ValaPrecompile)
12 # Add all your vala files and requires packages to the List below to include them in the build
13
14=== modified file 'src/Widgets/MprisGui.vala'
15--- src/Widgets/MprisGui.vala 2016-11-06 18:57:14 +0000
16+++ src/Widgets/MprisGui.vala 2016-11-07 15:14:30 +0000
17@@ -443,8 +443,7 @@
18 title_label.set_markup ("<b>%s</b>".printf (Markup.escape_text (title)));
19
20 if ("xesam:artist" in metadata && metadata["xesam:artist"].is_of_type (VariantType.STRING_ARRAY)) {
21- /* get_strv causes a segfault from multiple free's on vala's side. */
22- string[] artists = metadata["xesam:artist"].dup_strv ();
23+ (unowned string)[] artists = metadata["xesam:artist"].get_strv ();
24 artist_label.set_text (_("by ")+string.joinv (", ", artists));
25 } else {
26 if (client.player.playback_status == "Playing") {

Subscribers

People subscribed via source and target branches

to all changes: