[PATCH v3 2/9] parse-time-string: add a date/time parser to notmuch

2012-10-03 Thread Jani Nikula
On Tue, 25 Sep 2012, Michal Sojka wrote: > Hello Jani, > > On Wed, Sep 12 2012, Jani Nikula wrote: >> Add a date/time parser to notmuch, to be used for adding date range >> query support for notmuch lib later on. Add the parser to a directory >> of its own to make it independent of the rest of the

[PATCH v3 2/9] parse-time-string: add a date/time parser to notmuch

2012-10-03 Thread Michal Sojka
On Wed, Oct 03 2012, Jani Nikula wrote: > On Tue, 25 Sep 2012, Michal Sojka wrote: >> Hello Jani, >> >> On Wed, Sep 12 2012, Jani Nikula wrote: >>> Add a date/time parser to notmuch, to be used for adding date range >>> query support for notmuch lib later on. Add the parser to a directory >>> of i

Re: [PATCH v3 2/9] parse-time-string: add a date/time parser to notmuch

2012-10-03 Thread Michal Sojka
On Wed, Oct 03 2012, Jani Nikula wrote: > On Tue, 25 Sep 2012, Michal Sojka wrote: >> Hello Jani, >> >> On Wed, Sep 12 2012, Jani Nikula wrote: >>> Add a date/time parser to notmuch, to be used for adding date range >>> query support for notmuch lib later on. Add the parser to a directory >>> of i

[PATCH v3 2/9] parse-time-string: add a date/time parser to notmuch

2012-09-25 Thread Michal Sojka
Hello Jani, On Wed, Sep 12 2012, Jani Nikula wrote: > Add a date/time parser to notmuch, to be used for adding date range > query support for notmuch lib later on. Add the parser to a directory > of its own to make it independent of the rest of the notmuch code > base. First of all, thank you ver

[PATCH v3 2/9] parse-time-string: add a date/time parser to notmuch

2012-09-17 Thread Jani Nikula
On Mon, 17 Sep 2012, Michal Nazarewicz wrote: > Bison can do a lot of weird stuff including modifying how lexer > interpretes tokens even while parsing given grammar rule. I'll just have to take your word for it. > I'm sorry. I sometime tend to go into extremes with my statements, so > yes, the

[PATCH v3 2/9] parse-time-string: add a date/time parser to notmuch

2012-09-17 Thread Michal Nazarewicz
> On Thu, 13 Sep 2012, Michal Nazarewicz wrote: >> Have you consider doing the same in bison? I consider the code totally >> unreadable and unmaintainable. On Thu, Sep 13 2012, Jani Nikula wrote: > I do not think you could easily do everything that this parser does in > bison. But then I'm not a

Re: [PATCH v3 2/9] parse-time-string: add a date/time parser to notmuch

2012-09-17 Thread Jani Nikula
On Mon, 17 Sep 2012, Michal Nazarewicz wrote: > Bison can do a lot of weird stuff including modifying how lexer > interpretes tokens even while parsing given grammar rule. I'll just have to take your word for it. > I'm sorry. I sometime tend to go into extremes with my statements, so > yes, the

Re: [PATCH v3 2/9] parse-time-string: add a date/time parser to notmuch

2012-09-17 Thread Michal Nazarewicz
> On Thu, 13 Sep 2012, Michal Nazarewicz wrote: >> Have you consider doing the same in bison? I consider the code totally >> unreadable and unmaintainable. On Thu, Sep 13 2012, Jani Nikula wrote: > I do not think you could easily do everything that this parser does in > bison. But then I'm not a

[PATCH v3 2/9] parse-time-string: add a date/time parser to notmuch

2012-09-13 Thread Tomi Ollila
On Thu, Sep 13 2012, Michal Nazarewicz wrote: > On Wed, Sep 12 2012, Jani Nikula wrote: >> Add a date/time parser to notmuch, to be used for adding date range >> query support for notmuch lib later on. Add the parser to a directory >> of its own to make it independent of the rest of the notmuch

[PATCH v3 2/9] parse-time-string: add a date/time parser to notmuch

2012-09-13 Thread Jani Nikula
On Thu, 13 Sep 2012, Michal Nazarewicz wrote: > On Wed, Sep 12 2012, Jani Nikula wrote: >> Add a date/time parser to notmuch, to be used for adding date range >> query support for notmuch lib later on. Add the parser to a directory >> of its own to make it independent of the rest of the notmuch c

[PATCH v3 2/9] parse-time-string: add a date/time parser to notmuch

2012-09-13 Thread Michal Nazarewicz
On Wed, Sep 12 2012, Jani Nikula wrote: > Add a date/time parser to notmuch, to be used for adding date range > query support for notmuch lib later on. Add the parser to a directory > of its own to make it independent of the rest of the notmuch code > base. > > Signed-off-by: Jani Nikula Have yo

Re: [PATCH v3 2/9] parse-time-string: add a date/time parser to notmuch

2012-09-13 Thread Tomi Ollila
On Thu, Sep 13 2012, Michal Nazarewicz wrote: > On Wed, Sep 12 2012, Jani Nikula wrote: >> Add a date/time parser to notmuch, to be used for adding date range >> query support for notmuch lib later on. Add the parser to a directory >> of its own to make it independent of the rest of the notmuch

Re: [PATCH v3 2/9] parse-time-string: add a date/time parser to notmuch

2012-09-13 Thread Jani Nikula
On Thu, 13 Sep 2012, Michal Nazarewicz wrote: > On Wed, Sep 12 2012, Jani Nikula wrote: >> Add a date/time parser to notmuch, to be used for adding date range >> query support for notmuch lib later on. Add the parser to a directory >> of its own to make it independent of the rest of the notmuch c

Re: [PATCH v3 2/9] parse-time-string: add a date/time parser to notmuch

2012-09-13 Thread Michal Nazarewicz
On Wed, Sep 12 2012, Jani Nikula wrote: > Add a date/time parser to notmuch, to be used for adding date range > query support for notmuch lib later on. Add the parser to a directory > of its own to make it independent of the rest of the notmuch code > base. > > Signed-off-by: Jani Nikula Have yo

[PATCH v3 2/9] parse-time-string: add a date/time parser to notmuch

2012-09-13 Thread Jani Nikula
Add a date/time parser to notmuch, to be used for adding date range query support for notmuch lib later on. Add the parser to a directory of its own to make it independent of the rest of the notmuch code base. Signed-off-by: Jani Nikula --- Makefile |2 +- parse-