dee

Merge lp:~mhr3/dee/fix-slist-iteration into lp:dee

  1. fix-slist-iteration
  2. Merge into trunk
Proposed by Michal Hruby on 2012-12-03
Status: Merged
Approved by: Sebastien Bacher on 2012-12-03
Approved revision: 391
Merged at revision: 391
Proposed branch: lp:~mhr3/dee/fix-slist-iteration
Merge into: lp:dee
Diff against target: 12 lines (+1/-1)
1 file modified
src/dee-serializable.c (+1/-1)
To merge this branch: bzr merge lp:~mhr3/dee/fix-slist-iteration
Reviewer Review Type Date Requested Status
Sebastien Bacher 2012-12-03 Approve on 2012-12-03
PS Jenkins bot continuous-integration Pending
Review via email: mp+137638@code.launchpad.net

Commit Message

Fix incorrect iteration

Description of the Change

Fix incorrect iteration

To post a comment you must log in.
Sebastien Bacher (seb128) wrote :

Thanks, looks good and fix the segfault issue I was having

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/dee-serializable.c'
2--- src/dee-serializable.c 2012-11-23 17:41:41 +0000
3+++ src/dee-serializable.c 2012-12-03 17:26:44 +0000
4@@ -195,7 +195,7 @@
5
6 /* Find the right parser and apply it */
7 parsers = g_hash_table_lookup (parsers_by_gtype, gtype_name);
8- for (iter = parsers; iter != NULL; iter++)
9+ for (iter = parsers; iter != NULL; iter = iter->next)
10 {
11 parser = (Parser *) iter->data;
12 if (g_variant_type_equal (parser->vtype, vtype))

Subscribers

People subscribed via source and target branches

to all changes: