Hi,

On Tuesday 01 June 2004 09:57, Graham Barr wrote:
> I like the idea but,
>
> In your implementation you forgot to shift the fist argument from @_,
> so Authen::SASL would get pushed into the array and it should not.
>
> It could be argued that if none of the mentioned packages are loadable
> then the import should die, not just silently continue to use the
> default.
>
> It makes the use line very wordy, I would suggest that any argument
> passed that does not contain :: is prefixed with Authen::SASL:: by the
> code.

I did not get 'round to correcting my patch until now.
But now it is not necessary any more since you already did it.
(and you did it in less code than I would have needed ;-)

Thanks 
Peter

-- 
Peter Marschall
eMail: [EMAIL PROTECTED]

Reply via email to