On Jan 15, 2008, at 9:33 AM, [EMAIL PROTECTED] wrote:
I think the commits where FIXMEs and TODOs are just erased should be
rolled
back except in those cases where the FIXME or TODO does not add
value or
the issue pointed out by the FIXME or TODO was fixed. Does anybody
else
have this opinion or am I making too much out of nothing here?
I am not a committer - but my general opinion on these things is to
leave them in 100% unless someone has carefully analyzed each one and
produced a solution or made sure that they could be deleted.
There is no harm in having some TODOs and FIXMEs sitting there in
trunk - even for quite some time. Their very existence or the
information that they have will help some programmer in the future who
stumbles across a big or issue.
I say leave them in and take them out slowly thinking about each one.
If there is a whole class - (like the printstack ones) go ahead and
fix en masse.
But don't just remove en-masse IMO.
Chuck Severance
