Merge lp:~ivaldi/midori/devpet-backtrace-vapi into lp:midori

Proposed by André Stösel
Status: Merged
Approved by: Cris Dywan
Approved revision: 6300
Merged at revision: 6314
Proposed branch: lp:~ivaldi/midori/devpet-backtrace-vapi
Merge into: lp:midori
Diff against target: 30 lines (+10/-1)
2 files modified
extensions/devpet.vala (+2/-1)
midori/midori.vapi (+8/-0)
To merge this branch: bzr merge lp:~ivaldi/midori/devpet-backtrace-vapi
Reviewer Review Type Date Requested Status
Cris Dywan Approve
Review via email: mp+178424@code.launchpad.net

Commit message

Added fixed version of backtrace_symbols to midori vapi

Description of the change

This fixes build errors with older vala versions

To post a comment you must log in.
Revision history for this message
Rico Tzschichholz (ricotz) wrote :
Revision history for this message
André Stösel (ivaldi) wrote :

Hm... funny, Christian and I talked about this in jabber and we both wondered why they changed it...

Thank you very much!

Revision history for this message
Cris Dywan (kalikiana) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'extensions/devpet.vala'
2--- extensions/devpet.vala 2013-08-02 09:15:21 +0000
3+++ extensions/devpet.vala 2013-08-03 21:06:28 +0000
4@@ -178,7 +178,8 @@
5 string bt = "";
6 void* buffer[100];
7 int num = Linux.backtrace (buffer, 100);
8- unowned string[] symbols = Linux.backtrace_symbols (buffer, num);
9+ /* Upstream bug: https://git.gnome.org/browse/vala/commit/?id=f402af94e8471c8314ee7a312260a776e4d6fbe2 */
10+ unowned string[] symbols = Midori.Linux.backtrace_symbols (buffer, num);
11 if (symbols != null) {
12 /* we don't need the first three lines */
13 for (int i = 3; i < num; i++) {
14
15=== modified file 'midori/midori.vapi'
16--- midori/midori.vapi 2013-08-02 23:45:16 +0000
17+++ midori/midori.vapi 2013-08-03 21:06:28 +0000
18@@ -258,5 +258,13 @@
19 public static void create_win32_desktop_lnk (string prefix, string filename, string uri);
20 #endif
21 }
22+
23+ #if !HAVE_WIN32
24+ [CCode (lower_case_cprefix = "")]
25+ namespace Linux {
26+ [CCode (cheader_filename = "execinfo.h", array_length = false)]
27+ public unowned string[] backtrace_symbols (void* buffer, int size);
28+ }
29+ #endif
30 }
31

Subscribers

People subscribed via source and target branches

to all changes: