#3144: Parent and child match pattern modifier
--------------------------+----------------------
  Reporter:  jlh          |      Owner:  mutt-dev
      Type:  enhancement  |     Status:  new
  Priority:  minor        |  Milestone:
 Component:  mutt         |    Version:
Resolution:               |   Keywords:
--------------------------+----------------------

Comment (by kevin8t8):

 I've tested it more, and it seems to work properly.

 One thing I was concerned about was whether thread->child->prev could be
 non-null.  The original patch checked for that case, and the
 match_threadcomplete() function marks to go prev and next when traversing
 down.

 However, I've looked through the thread.c code, and it does not seem to be
 the case that thread->child ever points to the middle of the children.
 You can see this in linearize_tree(), mutt_sort_subthreads(),
 mutt_sort_threads() and the way in which insert_message() is called, and
 also in _mutt_traverse_thread().

 If it turns out I've missed something, then I'll change the new functions,
 but I'm not going to add voodoo code just in case.

 I'm uploading a v2 patch with documentation.  Unless there are any
 objections, I'll push it up in a few days.

--
Ticket URL: <https://dev.mutt.org/trac/ticket/3144#comment:15>
Mutt <http://www.mutt.org/>
The Mutt mail user agent

Reply via email to