Regarding the logging comment/question on a previous [pull-request](https://github.com/ModeShape/modeshape/pull/654#commitcomment-2475446), this line is a good example of one that I would wrap in a `if (TRACE) {...}` block. Note how the `((TextExtractors) extractor).size()` operation is _always_ performed, even if trace logging is not enabled.

The TRACE logging calls (on lines 664, 702, 719, 725 and 728) are all okay because existing values/constants are passed in, so there actually is no overhead.