@adamralph Good points.

I think we could make it work with the original design.  One option would be passing that LineProcessors collection into the constructors of the different classes that need to be able to consume / use it (ModuleConfiguration for one).  If you go back far enough in the jabbr log (4/15/2014), you'll see @khellang discussing that issue so he might have some feedback on this.

If we do decide to make it work with something closer to the original design of the class, I might consider reforking and sending a new pull request at that point instead of adding more commits to this one.  What are your thoughts on that?