Re: [log4perl-devel] [PATCH] Config.pm breaks if $/ is not set correctly

[Jim C. <jc...@di...>]

Mike Schilli wrote:

>Kevin Goess wrote on 5/13/2004, 11:09 AM:
>
> > In which case our init will fail since the user has (however
> > ill-advisedly) tinkered with $/, and we are assuming $/ is still "\n"
>  
>

even if he properly localized it, the init-call could be inside the block,
and thus be broken.   I think its best to protect the user as his patch 
does.

>Right, that's the behaviour which the manpage discourages. I'm not sure 
>if modules typically catch this or are supposed to catch this. But if 
>you guys want it in, I'm ok with it if it doesn't break anything. If 
>you're sure that setting $/ = "\n" is compatible with all OSes, I'll 
>apply Dennis' patch.
>
>  
>
you might have probs with windows - \r\n might work in htat case.
tho im not a windoze guy.

>--
>-- Mike
>Mike Schilli
>m...@pe...
>
>  
>