Launchpad has imported 7 comments from the remote bug at
https://bugs.freedesktop.org/show_bug.cgi?id=88846.
If you reply to an imported comment from within Launchpad, your comment
will be sent to the remote bug automatically. Read more about
Launchpad's inter-bugtracker facilities at
https://help.launchpad.net/InterBugTracking.
On 2015-01-28T08:46:57+00:00 Albert Astals Cid wrote:
If the lambda fails the code leaves a MainLoop around that breaks things
(i.e. the Qt main glib loop).
I'm attaching the patch that fixes the problem, the generated code with
and the patch.
Reply at:
https://bugs.launchpad.net/ubuntu/+source/zeitgeist/+bug/1415374/comments/0
On 2015-01-28T08:48:09+00:00 Albert Astals Cid wrote:
Created attachment 112905
The patch
Add finally {} to try {} catch {}
Reply at:
https://bugs.launchpad.net/ubuntu/+source/zeitgeist/+bug/1415374/comments/1
On 2015-01-28T08:49:14+00:00 Albert Astals Cid wrote:
Created attachment 112906
Generated .c code if unpatched
Note that there's some return; error conditions where the loop is not
exited correctly with g_main_loop_quit (_data1_-mainloop);
Reply at:
https://bugs.launchpad.net/ubuntu/+source/zeitgeist/+bug/1415374/comments/2
On 2015-01-28T08:49:52+00:00 Albert Astals Cid wrote:
Created attachment 112907
Generated .c code if patched
Note that all the exit paths properly have a g_main_loop_quit
(_data1_-mainloop);
Reply at:
https://bugs.launchpad.net/ubuntu/+source/zeitgeist/+bug/1415374/comments/3
On 2015-01-28T08:51:06+00:00 Albert Astals Cid wrote:
Please if possible credit this to Albert Astals Cid
albert.ast...@canonical.com and not to Albert Astals Cid
aa...@kde.org I'm just using my kde.org account since I didn't feel
like creating a new one just for this.
Reply at:
https://bugs.launchpad.net/ubuntu/+source/zeitgeist/+bug/1415374/comments/4
On 2015-01-29T15:29:55+00:00 Albert Astals Cid wrote:
Created attachment 112950
The patch
Reply at:
https://bugs.launchpad.net/ubuntu/+source/zeitgeist/+bug/1415374/comments/6
On 2015-01-29T15:35:56+00:00 Albert Astals Cid wrote:
It's been commited.
Reply at:
https://bugs.launchpad.net/ubuntu/+source/zeitgeist/+bug/1415374/comments/7
** Changed in: zeitgeist
Status: Unknown = Fix Released
** Changed in: zeitgeist
Importance: Unknown = Medium
--
You received this bug notification because you are a member of DX
Packages, which is subscribed to zeitgeist in Ubuntu.
Matching subscriptions: dx-packages
https://bugs.launchpad.net/bugs/1415374
Title:
Log: Loop not properly cleaned up on failures in Log constructor
Status in Zeitgeist Framework:
Fix Released
Status in zeitgeist package in Ubuntu:
New
Bug description:
If the lambda fails the code leaves a MainLoop around that breaks
things (i.e. the Qt main glib loop).
Patches and stuff at
https://bugs.freedesktop.org/show_bug.cgi?id=88846
To manage notifications about this bug go to:
https://bugs.launchpad.net/zeitgeist/+bug/1415374/+subscriptions
--
Mailing list: https://launchpad.net/~dx-packages
Post to : dx-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~dx-packages
More help : https://help.launchpad.net/ListHelp
