Merge lp:~codygarver/pantheon-files/workaround-1252491 into lp:~elementary-apps/pantheon-files/trunk

Proposed by Cody Garver
Status: Merged
Approved by: Julián Unrrein
Approved revision: 1377
Merged at revision: 1378
Proposed branch: lp:~codygarver/pantheon-files/workaround-1252491
Merge into: lp:~elementary-apps/pantheon-files/trunk
Diff against target: 22 lines (+10/-1)
1 file modified
src/ProgressUIHandler.vala (+10/-1)
To merge this branch: bzr merge lp:~codygarver/pantheon-files/workaround-1252491
Reviewer Review Type Date Requested Status
Julián Unrrein (community) Approve
Review via email: mp+197277@code.launchpad.net

Commit message

Provide a custom binding to preserve backwards compatibility with a binding change to a libnotify function. Fixes bug #1252491.

To post a comment you must log in.
Revision history for this message
Julián Unrrein (junrrein) wrote :

Compiles and works with Vala 0.16, 0.20 and 0.22.1.

Code style is good too, so I'm approving this.

Thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/ProgressUIHandler.vala'
--- src/ProgressUIHandler.vala 2013-11-21 01:08:15 +0000
+++ src/ProgressUIHandler.vala 2013-11-30 02:20:34 +0000
@@ -63,8 +63,17 @@
63 this.notification_supports_persistence = server_has_persistence ();63 this.notification_supports_persistence = server_has_persistence ();
64 }64 }
6565
66#if !VALA_0_24
67 [CCode (cheader_filename = "libnotify/notify.h", cname = "notify_get_server_caps")]
68 extern static GLib.List<string> notify_get_server_caps ();
69#endif
70
66 private bool server_has_persistence () {71 private bool server_has_persistence () {
67 unowned List<string> capabilities = Notify.get_server_caps ();72#if VALA_0_24
73 var capabilities = Notify.get_server_caps ();
74#else
75 var capabilities = notify_get_server_caps ();
76#endif
6877
69 return capabilities.find ("persistence") != null;78 return capabilities.find ("persistence") != null;
70 }79 }

Subscribers

People subscribed via source and target branches

to all changes: