As in, "What not to do?" {grin}

Let's see:

• The "lincoln loader" instead of SelfLoader: check
• Custom importer instead of Exporter: check
• Funky "is this a method or a subroutine" repeated everywhere: check
• Funky "args preceded by dash and/or args inside a hashref": check
• Odd mix of globals and/or use parameters and/or object instance vars to determine behavior: check

Admittedly, some of this stuff was created before the core functionality was available. But there's some mighty scary code in there.

In Lincoln's defense, this code is probably the second most frequently invoked module, after strict, and has had to deal with a lot of legacy issues.