(tye)Re2: Module Design Review

I'd just quote them.

You could make erroring out on undef in that case an option. If not, I would have to customize your module in order to use it with my existing module.

A few months back I read that version strings had been declared a worse abomination than pseudo hashes and had already been deprecated. Searching for verification, that doesn't appear to be the case.

Update: My existing module is Win32::TieRegistry and a typical use line looks like:

    my $Reg;
    use Win32::TieRegistry 0.20 (
        TiedRef => \$Reg,
        Delimiter => "/", ArrayValues => 1,
        qw( :REG_ KEY_ALL_ACCESS ),
    );
[download]

where options that take an argument are simply recognized by their value, not by having a leading "-". I'd already hoped to use the version argument to disable some default behavior that I had decided was a mistake, so your module would be a welcome aid. (:

- tye

Comment on (tye)Re2: Module Design Review - Exporter::VA Download Code

Replies are listed 'Best First'.
Re: (tye)Re2: Module Design Review - Exporter::VA by John M. Dlugosz (Monsignor) on Oct 29, 2002 at 20:40 UTC
I see. You have option/value pairs and constants imported in the normal manner complete with :TAG forms, but no functions! I already updated my design (back to) permitting pragmatic imports without a leading underscore. If you meant to write a callback and forgot to return something, you'll figure it out when the import doesn't import anything, so that's hardly a silent mysterious problem, and verbose mode will tell you exactly what really happened so you can find it easily enough. I'd be honored if you'd tinker with it before the official release, to make sure it does what you need for TieRegistry. I think that's a great example of something more exotic then the plain global function import situation.	[reply]
Re: (tye)Re2: Module Design Review - Exporter::VA by John M. Dlugosz (Monsignor) on Oct 29, 2002 at 17:12 UTC
Tell me what your existing module does. I take it it doesn't use the Exporter module, but your own?	[reply]
Re^3: Module Design Review - Exporter::VA by Aristotle (Chancellor) on Oct 29, 2002 at 22:26 UTC
Excuse the pun, but I couldn't resist - maybe that should be Win32::TyeRegistry? ducks Ok, ok, I'll be good now. `:^)` Makeshifts last the longest.	[reply]