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

List:       koffice-devel
Subject:    Re: BCI todos
From:       Raphael Langerhorst <raphael-langerhorst () gmx ! at>
Date:       2004-03-15 18:20:28
Message-ID: 200403151920.29340.raphael-langerhorst () gmx ! at
[Download RAW message or body]

On Sunday 14 March 2004 20:50, David Faure wrote:
> On Sunday 14 March 2004 09:12, Raphael Langerhorst wrote:
> > Hi all,
> >
> > while looking through some files in /lib/kofficecore I found a few BCI
> > todos, most of them are NOT marked as "TODO" in any way. Should I...
> >
> > a) add a "TODO" string to these comments (so they can be found while
> > looking for TODOs),
> >
> > b) write them down in a list and send this list to this [mailing] list,
> >
> > c) of just leave them as they are?
>
> d) Make the obvious changes
> (merging methods with overloads to add parameters, remove old unused
> methods etc.)

Thanks both for the hints, I'll probably do both: I'll mark the ones I'm not 
so sure about as todos and the ones that are obvious I'll change right away. 
A patch might come on Friday so it fits into the schedule ;)

besides: have a look at http://bugs.kde.org/show_bug.cgi?id=77574
I also added a working patch to the bugreport so you just need to apply it. 
IMO it should be save although it surely wouldn't hurt if an expert looks it 
through.

Regards,
Raphael
_______________________________________________
koffice-devel mailing list
koffice-devel@mail.kde.org
https://mail.kde.org/mailman/listinfo/koffice-devel
[prev in list] [next in list] [prev in thread] [next in thread]