Subject: | debugging left in module? |
Thanks for this module. I love the functionality, although I wish it
could be implemented as a role/mix-in/plugin so that it could be more
easily paired with other Mechanize extensions.
Here's a small point that could be improved.
I see that the source code contains this line:
open( SAV, ">>c:/mechanize.log" ) and print( SAV $log ) and close( SAV
);
I presume this is a mistake, because the docs don't mention anything
about "mechanize.log", and the code would only work on Windows.
Since there's no error checking, the line could silently fail to log and
the behavior could go unnoticed without reading the source code.