<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><span style="font-family: monospace;" class="">Hi Richard,</span><br style="font-family: monospace;" class=""><br style="font-family: monospace;" class=""><font face="monospace" class="">Ping with rebased and cleaned up versions of the patches. Could you maybe just have a look at the way ModuleProvider is threaded through and see if there is anything about the design that you would feel uncomfortable about having to fix it in a more detailed post-commit review?</font><br style="font-family: monospace;" class=""><br style="font-family: monospace;" class=""><span style="font-family: monospace;" class="">-- adrian</span><div class=""><font face="monospace" class=""><br class=""></font><div class=""></div></div></body></html>