in reply to Devel::Cover, Testing, 100%?!

If you can mathematically show that the second validation step will never be called, then you have a redundant check. This is one of the odder things test coverage will show.

I'd remove it and let CVS keep the memory. You're not removing it to get to 100% test coverage ... you're removing it because it's a useless piece of code.

------
We are the carpenters and bricklayers of the Information Age.

Then there are Damian modules.... *sigh* ... that's not about being less-lazy -- that's about being on some really good drugs -- you know, there is no spoon. - flyingmoose

I shouldn't have to say this, but any code, unless otherwise stated, is untested

In Section Seekers of Perl Wisdom