[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: Proposed new KDateTime class
From:       Nicolas Goutte <nicolasg () snafu ! de>
Date:       2005-11-12 9:49:22
Message-ID: 200511121030.46543.nicolasg () snafu ! de
[Download RAW message or body]

On Friday 11 November 2005 15:58, Benjamin Meyer wrote:
> On Tuesday 08 November 2005 5:00 pm, David Jarvie wrote:
> > Now that the time zone classes have been updated, there is a need to be
> > able to handle date/times which have an associated time zone. I attach a
> > proposed new kdecore class, KDateTime, which represents a date/time with
> > an associated time zone. The aim is to make time zone handling as
> > automatic as possible when manipulating dates and times. Its interface is
> > very similar to QDateTime, but it is not inherited from QDateTime mainly
> > because QDateTime's methods are not virtual.
>
> You don't inherit from QDateTime so I recommend coping the api exactly when
> possible.
>
> > The code compiles, but I until I finish the test program is untested.
> > Note that if the new class is committed, the time zone classes will need
> > to be amended to handle the new class.
> >
> > Comments please. (Note that I am going away and won't be able to respond
> > for a few days.)
>
> Header simple comments:
>
> kill the @file, there is no point to it.

Personally I think that it is good to be able to access Doxygen comment by the 
file name. (That is especially true for files having stuff beside classes.)

(...)

Have a nice day!

[prev in list] [next in list] [prev in thread] [next in thread]