Re: Merging two functions in mrp-task-manager.c

[Matt Burkhardt <mlb imparisystems com>]

I say go for it!  It sounds like it will result in less code overall.

On Sun, 2008-01-20 at 16:39 +0100, Maurice van der Pot wrote:
> 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.
>
> _______________________________________________
> Planner-dev-list mailing list
> Planner-dev-list gnome org
> http://mail.gnome.org/mailman/listinfo/planner-dev-list

-- 
Matt Burkhardt, MSTM
President
Impari Systems, Inc.
Phone:  (301) 644-3911
mlb imparisystems com
http://www.imparisystems.com