Mailing-List: contact avalon-dev-help@jakarta.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Avalon Development" <avalon-dev@jakarta.apache.org>
Content-Type: text/plain;
  charset="iso-8859-1"
From: Peter Donald <donaldp@apache.org>
To: "Avalon Development" <avalon-dev@jakarta.apache.org>
Subject: Re: [LogKit][PATCH] 'method' field for PatternFormatter
Date: Fri, 27 Jul 2001 03:33:49 +1000
References: <E15P2Iw-0008Q1-0Y@anchor-post-34.mail.demon.net>
 <01072701554906.00795@helm.realityforge.org>
 <3B60533B.93D533F5@anyware-tech.com>
In-Reply-To: <3B60533B.93D533F5@anyware-tech.com>
MIME-Version: 1.0
Message-Id: <0107270333490E.00795@helm.realityforge.org>
Content-Transfer-Encoding: 8bit

On Fri, 27 Jul 2001 03:28, Sylvain Wallez wrote:
> Peter Donald a �crit :
> > On Thu, 26 Jul 2001 00:43, Sylvain Wallez wrote:
> > > Since there seems to be interest for it, the files are attached and I'd
> > > be happy to see the non-Cocoon specific part of them included in a
> > > future LogKit release.
> >
> > Okay I have rolled the changes you made to PatternFormatter into main
> > PatternFormatter so hopefully you don't need ExtensiblePatternFormatter
> > anymore.
>
> Kewl (notice : I learn your langage ;) 

yikes! ;)

> > If you could make sure I didn't stuff up anything that would be kewl ;)
>
> Things you don't have included from ExtensiblePatternFormatter but are
> necessary :
>
> - the constant MAX_TYPE (or give it another name if you want) is
> necessary for subclasses robustness against evolutions of this class. If
> subclasses must rely on "TYPE_PRIORITY" or "8" to know at which number
> to start their new types, they will be broken if you add
> "TYPE_KEWL_PATTERN = 9" !
>
> - PatternRun is protected, which is good because it should only be used
> by PatternFormatter and its subclasses. But PatterRun members must be
> public, otherwise PatternFormatter subclasses won't be able to access
> them.

oops. Fixed. I did an eyeball diff .. guess my eyes aren't quite as good as 
the program version ;)

> What's the purpose of getExtension() ? It's never called in the code.

It was part of an experiment in easier ways to extend PatternFormatter. It 
failed and I guess I forgot to remove all of it ;)

> Thanks for putting this in Avalon.

Thanks for submitting it ! 

Cheers,

Pete

*-----------------------------------------------------*
| "Faced with the choice between changing one's mind, |
| and proving that there is no need to do so - almost |
| everyone gets busy on the proof."                   |
|              - John Kenneth Galbraith               |
*-----------------------------------------------------*

---------------------------------------------------------------------
To unsubscribe, e-mail: avalon-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: avalon-dev-help@jakarta.apache.org