Merge lp:~larsu/ido/buildfix into lp:ido/15.10

Proposed by Lars Karlitski
Status: Merged
Approved by: Sebastien Bacher
Approved revision: 193
Merged at revision: 195
Proposed branch: lp:~larsu/ido/buildfix
Merge into: lp:ido/15.10
Diff against target: 11 lines (+1/-1)
1 file modified
src/idosourcemenuitem.c (+1/-1)
To merge this branch: bzr merge lp:~larsu/ido/buildfix
Reviewer Review Type Date Requested Status
Sebastien Bacher Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+273222@code.launchpad.net

Commit message

idosourcemenuitem: change misplaced g_return_val_if_fail()

Description of the change

idosourcemenuitem: change misplaced g_return_val_if_fail()

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Sebastien Bacher (seb128) wrote :

looks good, thanks

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/idosourcemenuitem.c'
2--- src/idosourcemenuitem.c 2013-08-13 16:50:26 +0000
3+++ src/idosourcemenuitem.c 2015-10-02 12:35:32 +0000
4@@ -201,7 +201,7 @@
5 item->timer_id = 0;
6 }
7
8- g_return_val_if_fail (g_variant_is_of_type (state, G_VARIANT_TYPE ("(uxsb)")), FALSE);
9+ g_return_if_fail (g_variant_is_of_type (state, G_VARIANT_TYPE ("(uxsb)")));
10
11 g_variant_get (state, "(ux&sb)", &count, &time, &str, NULL);
12

Subscribers

People subscribed via source and target branches