Siterer Christian Grobmeier <[email protected]>:

LoggerAppenderSkeleton extends from LoggerAppender. LoggerAppender is
not used somewhere else in the whole project.
Is there really a need to have both classes? I currently think this is
a bit overkill. Why not having the classes mixed together and just
name it LoggerAppender?

Yes, everything in LoggerAppenderSkeleton can be moved to LoggerAppender. Both are abstract classes and LoggerAppender::singleton() is just used in configurator's (LoggerPropertyConfigurator, LoggerDOMConfigurator and the upcoming LoggerPhpConfigurator).

Knut



Reply via email to