There really is no need for a class to have a Logger instance as an instance member. However, we sometimes do this to save multiple calls to obtain the Logger instance. These can just as easily use a static Logger instance for the class as an instance variable.

Note: This would be an easy task for a new committer or someone looking to learn more about the project. Simply use your favorite IDE to search for class members of the org.modeshape.common.util.Logger class, and change as many as you can find to use a static member instead.