Hi, >> I also think, that only one instance is needed of LoggerHierarchy. > > "Recent log4j releases support multiple hierarchy trees. This > enhancement allows each virtual host to possess its own copy of the > logger hierarchy." > http://logging.apache.org/log4j/1.2/manual.html - NDC section
I just read this one. I don't think this requirement is proper for Log4PHP. We don't have our classes shared within a VM, we have our classes only shared within a request. However, IF we would need multiple hierarchy trees, then we would have a problem when using the classic singleton defined with getInstance in LoggerHierarchy. Cause moving from a single variable in Logger to an array which holds the hierarchies should not be a big problem. Guess this is a point for removing LoggerHierarchy::getInstance. Christian
