[PATCH 1/4] Import date/time parser from GNU coreutils

Michal Sojka sojkam1 at fel.cvut.cz
Tue Jan 25 00:41:42 PST 2011


On Mon, 24 Jan 2011, Jameson Rollins wrote:
> On Sun, 23 Jan 2011 12:47:24 +0100, Michal Sojka <sojkam1 at fel.cvut.cz> wrote:
> > This function have quite a lot dependencies. We may reduce them later it
> > it is a problem.
> > ---
> >  lib/c-ctype.c  |  398 +++++++
> >  lib/c-ctype.h  |  297 +++++
> >  lib/getdate.c  | 3497 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> >  lib/getdate.h  |   22 +
> >  lib/getdate.y  | 1572 +++++++++++++++++++++++++
> >  lib/gettime.c  |   48 +
> >  lib/intprops.h |   83 ++
> >  lib/timespec.h |   39 +
> >  lib/verify.h   |  140 +++
> >  9 files changed, 6096 insertions(+), 0 deletions(-)
> >  create mode 100644 lib/c-ctype.c
> >  create mode 100644 lib/c-ctype.h
> >  create mode 100644 lib/getdate.c
> >  create mode 100644 lib/getdate.h
> >  create mode 100644 lib/getdate.y
> >  create mode 100644 lib/gettime.c
> >  create mode 100644 lib/gettime.h
> >  create mode 100644 lib/intprops.h
> >  create mode 100644 lib/timespec.h
> >  create mode 100644 lib/verify.h
> 
> Hi, Michal.  I don't fully understand what's going on here, but it seems
> like you're embedding code copies from somewhere else.  If that's the
> case, is there a reason that we would need to do that, rather than just
> linking against an external library?

Well, if the embedded code is available in a library, it would be
definitely better to just use the library. But the above code is
statically linked to things like `date` command and is not available
separately.

Most of the dependencies could be eliminated since they usually
replicate functionality which is available in modern C library and are
there only for compatibility reasons.

On the other hand, if anybody knows a better date parser, perhaps in a
separate library, let me know.

-Michal


More information about the notmuch mailing list