Redundant code

[Philip Downer]

I was having a look through the http.pm file today and came across this
Comment:

# The filltemplate code described below is not currently in use. It has
# been replaced by Template Toolkit

The filltemplate routine was then left in the code, this is rather
dangerous and confusing if, like myself you don't spot the comment
first.
Rather than leaving redundant code in the software, shouldn't it be
deleted or at least commented out?

I'm not looking at the latest nightly image just the 6.0 release, so it
may have been removed since.

Phil.

[dean]

Excellent point. I'll remove.

On Apr 5, 2005, at 3:23 AM, Philip Downer wrote:

> I was having a look through the http.pm file today and came across this
> Comment:
>
> # The filltemplate code described below is not currently in use. It
> has
> # been replaced by Template Toolkit
>
> The filltemplate routine was then left in the code, this is rather
> dangerous and confusing if, like myself you don't spot the comment
> first.
> Rather than leaving redundant code in the software, shouldn't it be
> deleted or at least commented out?
>
> I'm not looking at the latest nightly image just the 6.0 release, so it
> may have been removed since.
>
> Phil.
>
>