From: Kevin G. <ke...@go...> - 2003-06-11 16:53:35
|
OK, I just implemented that and a few other tweaks. It's probably precipitous to include them in your pending release, just to be conservative. I got rid of the warnings printed during 014ConfErrs.t about 'ambiguous use of require without parens, tweaked the error messages given for bad config files, and added hooks to JavaMap so users can add their own. Changes are checked in, a diff is attached if anybody's curious. Mike Schilli wrote: > On Mon, 9 Jun 2003, Kevin Goess wrote: > > >>That's a good point, but that solution gives Java programmers the same >>problem, if they want to make an appender in a simple namespace. >> >>How bout if we make it explicit, something like >> if ($Log::Log4perl::JavaMap::translate{$appenderclass}){ >> #then it's Java >> }else{ >> #it's Perl >> >>actually <type...type...type> like the attached? > > > I like it -- although I'm not sure if that's a likely case. After all, > someone would have to write a new Java class and most likely a Perl > class to map it to in order use it. > > >>Hmmm, but the user would need a way to add their custom class's name to >>the java map. I guess they could just do that before they call init(), >>I should document that, shouldn't I? Or maybe even make a real accessor >>method for the %translate hash, what do you think? > > > I'm fine either way, whatever works :). > > -- Mike > > Mike Schilli > log...@pe... > http://perlmeister.com > http://log4perl.sourceforge.net -- Happy Trails . . . Kevin M. Goess (and Anne and Frank) 904 Carmel Ave. Albany, CA 94706 (510) 525-5217 |