From koffice-devel Mon Mar 15 18:20:28 2004 From: Raphael Langerhorst Date: Mon, 15 Mar 2004 18:20:28 +0000 To: koffice-devel Subject: Re: BCI todos Message-Id: <200403151920.29340.raphael-langerhorst () gmx ! at> X-MARC-Message: https://marc.info/?l=koffice-devel&m=107937494905105 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