Re: [Planner Dev] "Nonstandard Days" and "Dominant tasks" - 4th patch - style corrections



On Sun, 2006-03-19 at 18:51 +0100, Matteo Nastasi wrote:
> On Sun, Mar 19, 2006 at 11:10:12AM -0500, Kurt Maute wrote:
> > On Fri, 2006-03-17 at 14:10 +0100, Matteo Nastasi wrote:
> > 
> > > > > There are other kinds of corrections to made ?
> > 
> > The only thing I noticed is that we prefer to initialize variables
> > outside their declaration statements - just makes the code a bit more
> > readable.
> Ok, I will adjust this.

excellent.


> If there aren't problems with this solution: can I "ifdeffing" the ypad
> related part of code to don't lose my work (maybe gtk team solve their
> problem) or must I remove it ?

I think the best solution to this is to save that code in a patch file.
Ifdef's should be kept to a minimum.

Thanks for all your work on this, Matteo.

-- 
Kurt Maute <kurt maute us>




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]