reference loop fix and tests, v2

Subject: reference loop fix and tests, v2

Date: Fri, 13 Apr 2018 22:46:05 -0300

To: notmuch@notmuchmail.org

Cc:

From: David Bremner

Here's a tidied version of the reference loop fix. There's actually
only one non-comment line of code changed. As you can guess from the
NEWS patch, I'm planning a point release to include this bugfix. I'd
also like to include the mset fix [1], so it would be great if someone
could test that.

[1]: id:20180406114307.7067-1-david@tethera.net

_______________________________________________
notmuch mailing list
notmuch@notmuchmail.org
https://notmuchmail.org/mailman/listinfo/notmuch

• Previous message (by thread): Re: reference loop fix and tests, v2

Thread:

• David Bremner—reference loop fix and tests, v2 [inbox, unread]
  • David Bremner—[PATCH 1/5] test: two new messages for the 'broken' corpus [inbox, notmuch::obsolete, notmuch::patch, unread]
    • Tomi Ollila—Re: [PATCH 1/5] test: two new messages for the 'broken' corpus [inbox, unread]
      • Daniel Kahn Gillmor—Re: [PATCH 1/5] test: two new messages for the 'broken' corpus [inbox, unread]
  • David Bremner—[PATCH 2/5] test: add known broken test for indexing an In-Reply-To loop. [inbox, notmuch::obsolete, notmuch::patch, unread]
  • David Bremner—[PATCH 3/5] lib: break reference loop by choosing arbitrary top level msg [inbox, notmuch::obsolete, notmuch::patch, unread]
    • Daniel Kahn Gillmor—Re: [PATCH 3/5] lib: break reference loop by choosing arbitrary top level msg [inbox, unread]
  • David Bremner—[PATCH 4/5] test: add second reference loop test [inbox, notmuch::obsolete, notmuch::patch, unread]
  • David Bremner—[PATCH 5/5] NEWS: add item for reference loop fix. [inbox, notmuch::obsolete, notmuch::patch, unread]
    • David Bremner—v3 reference loop fix [inbox, unread]
      • David Bremner—[PATCH 1/6] test: two new messages for the 'broken' corpus [inbox, notmuch::patch, notmuch::pushed, unread]
      • David Bremner—[PATCH 2/6] test: add known broken test for indexing an In-Reply-To loop. [inbox, notmuch::patch, notmuch::pushed, unread]
      • David Bremner—[PATCH 3/6] lib: break reference loop by choosing arbitrary top level msg [inbox, notmuch::patch, notmuch::pushed, unread]
      • David Bremner—[PATCH 4/6] NEWS: add item for reference loop fix. [inbox, notmuch::patch, notmuch::pushed, unread]
      • David Bremner—[PATCH 5/6] test: add known broken test for thread ordering from a loop [inbox, notmuch::patch, notmuch::pushed, unread]
      • David Bremner—[PATCH 6/6] lib: choose oldest message when breaking reference loops [inbox, notmuch::patch, notmuch::pushed, unread]
        • Tomi Ollila—Re: [PATCH 6/6] lib: choose oldest message when breaking reference loops [inbox, unread]
          • David Bremner—Re: [PATCH 6/6] lib: choose oldest message when breaking reference loops [inbox, unread]
            • Tomi Ollila—Re: [PATCH 6/6] lib: choose oldest message when breaking reference loops [inbox, unread]
  • David Bremner—Re: reference loop fix and tests, v2 [inbox, unread]