I don't think the $SWAP global is the right implementation. It's too global :-)

What about having

alias $original => $alias;
[download]

Alias $alias => $original;
[download]

Thanks, Jenda
Always code as if the guy who ends up maintaining your code will be a violent psychopath who knows where you live.
   -- Rick Osborne

Edit by castaway: Closed small tag in signature

That's what local is for, no?

Makeshifts last the longest.

Definitely no. If I did

sub foo {
 local $SWAP = 1;
 alias $alias => $original;
 ...
 Module::bar(some,params);
 ...
}
[download]

and someone else in the Module writes

sub bar {
 alias $original => $alias;
 ...
}
[download]

And suddenly his/her code starts trying to do something else than it should. Just because it was called from my foo() instead of directly. The local() could only "help" if I used it like this:

sub foo {
 {local $SWAP = 1;alias $alias => $original;}
 ...
}
[download]

But that's to long. I'd need the $SWAP to be lexical to my whole file, not local to the current call/block stack.

I think the nicest solution is alias $original => $alias; and Alias $alias => $original;.

Or maybe I should just learn it's the other way around versus what I'd expect :-)

Jenda
Always code as if the guy who ends up maintaining your code will be a violent psychopath who knows where you live.
   -- Rick Osborne

Edit by castaway: Closed small tag in signature