[boost] Comments on date/time library

From: Steven Kirk (steven.kirk_at_[hidden])
Date: 2003-01-17 08:12:51

Hi, I have just begun to use the boost date/time library and would like to
make some comments:

Default Construction
--------------------
It would be useful if default constructors for gregorian::date and
posix_time::ptime were provided. The default constructors could maybe work
as follows:

gregorian::date() ==>   gregorian::date(not_a_date_time)
posix_time::ptime() ==> posix_time::ptime(gregorian::date(not_a_date_time),
                                          time_duration(0, 0, 0))

DATE_TIME_INLINE
----------------
It seems wrong that when using the default build of the date/time library, I
have to #define DATE_TIME_INLINE in all compilation units that use the
date/time library. Wouldn't it be better to reverse this logic and define
DATE_TIME_NO_INLINE only when a non-default build is required?

Also, shouldn't this symbol be called BOOST_DATE_TIME_INLINE (or
BOOST_DATE_TIME_NO_INLINE :)

Building
--------
When building the current CVS state with Borland C++ Builder 5, I am still
getting all tests and examples built when I only want the library built.

Cheers
S.

______________________________________
Inflex - installed on mailserver for domain @status-scientific.com
Queries to: postmaster_at_[hidden]