[configuration] ConfigurationFactory

[Emmanuel Bourg <sm...@lfjr.net>]

Is there a need to keep the inner classes AdditionalConfigurationData 
and ConfigurationBuilder in ConfigurationFactory static ? I also wonder 
if it is desirable to have the other inner classes public, I would make 
them private or package visible.

Emmanuel Bourg


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

Re: [configuration] ConfigurationFactory

[Oliver Heger <he...@med.uni-marburg.de>]

There is no special need that these classes are static. I prefer to make 
inner classes static if they do not need a reference to their outer 
class and are merely utility classes.

The public inner classes will be called and instantiated by Digester 
when the XML file is parsed. I think this forces at least some of them 
being public.

Oliver

Emmanuel Bourg wrote:

> Is there a need to keep the inner classes AdditionalConfigurationData 
> and ConfigurationBuilder in ConfigurationFactory static ? I also wonder 
> if it is desirable to have the other inner classes public, I would make 
> them private or package visible.
> 
> Emmanuel Bourg
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: commons-dev-help@jakarta.apache.org
> 
> 

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