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

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

Date: Mon, 17 Sep 2012 13:10:50 -0300

To: Michal Nazarewicz, notmuch@notmuchmail.org

Cc:

From: David Bremner

Michal Nazarewicz <mina86@mina86.com> writes:

> On Thu, Sep 13 2012, David Bremner wrote:
>> I don't find ranges being half-open by default to be very
>> intuitive. Perhaps I don't program in python enough.
[snip]
> 	date:2012/01/01..2012/01/01 + 1 day
>
> in my opinion, that should give results from the first of January only,
> since “+ 1 day” indicates in a way how long user want the period to be.

Sorry, still not convinced.  My comment about python was more or less a
joke; I get the analogy with C, but (1) I don't think C is a reasonable
UI guide, and (2), at least in the C case, the half openness of the
range is explicitly given by the operators used.

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]