Re: [PATCH v3 6/9] lib: add date range query support

Subject: Re: [PATCH v3 6/9] lib: add date range query support

Date: Thu, 13 Sep 2012 09:32:30 -0300

To: Michal Nazarewicz, Jani Nikula, notmuch@notmuchmail.org

Cc:

From: David Bremner

Michal Nazarewicz <mina86@mina86.com> writes:

> IMO this is totally unintuitive and not how the range should work.
> date:foo..bar should return messages whose date >= foo and < bar.  So
> for instance date:november..yesterday should return messages whose date
> is > 2012/11/01 00:00:00 and < 2012/09/12 00:00:00.  So to get
> yesterdays messages one would do: date:yesterday..today.

I don't find ranges being half-open by default to be very
intuitive. Perhaps I don't program in python enough.

d

• Previous message (by thread): Re: [PATCH] test: Improve tests for the date/time parser module

Thread:

• Jani Nikula—[PATCH v3 0/9] notmuch search date:since..until query support [inbox, unread]
  • Jani Nikula—[PATCH v3 1/9] build: drop the -Wswitch-enum warning [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Jani Nikula—[PATCH v3 2/9] parse-time-string: add a date/time parser to notmuch [inbox, notmuch::obsolete, notmuch::patch, unread]
    • Michal Nazarewicz—Re: [PATCH v3 2/9] parse-time-string: add a date/time parser to notmuch [inbox, signed, unread]
      • Jani Nikula—Re: [PATCH v3 2/9] parse-time-string: add a date/time parser to notmuch [inbox, unread]
        • Michal Nazarewicz—Re: [PATCH v3 2/9] parse-time-string: add a date/time parser to notmuch [inbox, signed, unread]
          • Jani Nikula—Re: [PATCH v3 2/9] parse-time-string: add a date/time parser to notmuch [inbox, unread]
      • Tomi Ollila—Re: [PATCH v3 2/9] parse-time-string: add a date/time parser to notmuch [inbox, unread]
    • Michal Sojka—Re: [PATCH v3 2/9] parse-time-string: add a date/time parser to notmuch [inbox, unread]
      • Jani Nikula—Re: [PATCH v3 2/9] parse-time-string: add a date/time parser to notmuch [inbox, unread]
        • Michal Sojka—Re: [PATCH v3 2/9] parse-time-string: add a date/time parser to notmuch [inbox, unread]
  • Jani Nikula—[PATCH v3 3/9] test: add new test tool parse-time for date/time parser [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Jani Nikula—[PATCH v3 4/9] test: add smoke tests for the date/time parser module [inbox, notmuch::obsolete, notmuch::patch, unread]
    • Michal Sojka—[PATCH] test: Improve tests for the date/time parser module [inbox, unread]
      • Jani Nikula—Re: [PATCH] test: Improve tests for the date/time parser module [inbox, unread]
        • Michal Sojka—Re: [PATCH] test: Improve tests for the date/time parser module [inbox, unread]
  • Jani Nikula—[PATCH v3 5/9] build: build parse-time-string as part of the notmuch lib and static cli [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Jani Nikula—[PATCH v3 6/9] lib: add date range query support [inbox, notmuch::obsolete, notmuch::patch, unread]
    • Michal Nazarewicz—Re: [PATCH v3 6/9] lib: add date range query support [inbox, signed, unread]
      • Tomi Ollila—Re: [PATCH v3 6/9] lib: add date range query support [inbox, unread]
        • Jani Nikula—Re: [PATCH v3 6/9] lib: add date range query support [inbox, unread]
          • Michal Nazarewicz—Re: [PATCH v3 6/9] lib: add date range query support [inbox, signed, unread]
      • David Bremner—Re: [PATCH v3 6/9] lib: add date range query support [inbox, unread]
        • Michal Nazarewicz—Re: [PATCH v3 6/9] lib: add date range query support [inbox, signed, unread]
          • Tomi Ollila—Re: [PATCH v3 6/9] lib: add date range query support [inbox, unread]
          • David Bremner—Re: [PATCH v3 6/9] lib: add date range query support [inbox, unread]
      • Michal Sojka—Re: [PATCH v3 6/9] lib: add date range query support [inbox, unread]
  • Jani Nikula—[PATCH v3 7/9] test: add tests for date:since..until range queries [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Jani Nikula—[PATCH v3 8/9] man: document the date:since..until range queries [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Jani Nikula—[PATCH v3 9/9] NEWS: date range search support [inbox, notmuch::obsolete, notmuch::patch, unread]