This is an automated email from the git hooks/post-receive script.

guillem pushed a commit to branch master
in repository dpkg.

View the commit online:
https://git.dpkg.org/cgit/dpkg/dpkg.git/commit/?id=484ae17976645e2eb48a3f5d0d56e2b1dca63a6a

commit 484ae17976645e2eb48a3f5d0d56e2b1dca63a6a
Author: Guillem Jover <[email protected]>
AuthorDate: Tue Nov 6 03:01:32 2018 +0100

    dpkg: Use common pattern of assigning as an iterator
    
    Assign hare->next to hare instead of using the temporary variable, so
    that it's quicker to spot as an iterator assignment.
---
 debian/changelog | 1 +
 src/trigproc.c   | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/debian/changelog b/debian/changelog
index 525600048..b254ffacc 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -79,6 +79,7 @@ dpkg (1.19.3) UNRELEASED; urgency=medium
     - update-alternatives: Switch verbose selection into an enum.
     - dpkg: Negate tortoise_not_in_hare() function name and return value.
     - dpkg: Initialize trigcyclenode's next member once.
+    - dpkg: Use common pattern of assigning as an iterator.
   * Build system:
     - get-version: Use a format string with printf.
     - run-script: Use $() instead of deprecated ``.
diff --git a/src/trigproc.c b/src/trigproc.c
index 7cbf23c4a..3c670f8b1 100644
--- a/src/trigproc.c
+++ b/src/trigproc.c
@@ -288,7 +288,7 @@ check_trigger_cycle(struct pkginfo *processing_now)
        }
 
        hare->next = tcn;
-       hare = tcn;
+       hare = hare->next;
        if (tortoise_advance)
                tortoise = tortoise->next;
        tortoise_advance = !tortoise_advance;

-- 
Dpkg.Org's dpkg

Reply via email to