While working on a fix for some bug I noticed that task_manager_calculate_task_finish and task_manager_calculate_start_from_finish are incredibly similar and that the duplication is causing some problems. Some bugs that are present in both functions are being fixed in only one of them. Also features are sometimes added to one and forgotten in the other. See here for a diff (generated with kdiff3 using manual diff alignments): http://www.kfk4ever.com/~griffon26/shared/plannerdiff.pdf My proposal is to merge these functions into one and adding a parameter to select the behaviour. This may result in some ifs, but I'd rather have that than the current code duplication. Thoughts? Maurice. -- Maurice van der Pot Gentoo Linux Developer griffon26 gentoo org http://www.gentoo.org Gnome Planner Developer griffon26 kfk4ever com http://live.gnome.org/Planner
Attachment:
pgpYJ1c0epqAT.pgp
Description: PGP signature