Hi Argyrios,<br><br><div class="gmail_quote">On Mon, Nov 5, 2012 at 10:08 PM, Argyrios Kyrtzidis <span dir="ltr"><<a href="mailto:kyrtzidis@apple.com" target="_blank">kyrtzidis@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word"><div><div class="im"><div>On Nov 5, 2012, at 12:05 PM, Kim Gräsman <<a href="mailto:kim.grasman@gmail.com" target="_blank">kim.grasman@gmail.com</a>> wrote:</div><blockquote type="cite">
<div class="gmail_quote">On Mon, Nov 5, 2012 at 8:50 PM, Douglas Gregor <span dir="ltr"><<a href="mailto:dgregor@apple.com" target="_blank">dgregor@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<div><br>
On Nov 5, 2012, at 11:44 AM, Argyrios Kyrtzidis <<a href="mailto:kyrtzidis@apple.com" target="_blank">kyrtzidis@apple.com</a>> wrote:<br>
<br>
> Could we change the Preprocessor to accept an optional pointer to a ModuleLoader ?<br></div></blockquote><div><br></div><div>Since Preprocessor has a getModuleLoader() method, that would force null-checks on all callers. Currently only 2 of them, so no big deal, but I tend to shy away from exposing NULLs.</div>
</div></blockquote><div><br></div></div><div>How about if you don't pass an implementation of ModuleLoader, the preprocessor creates internally the "VoidModuleLoader".</div></div></div></blockquote><div><br>
</div><div>I tried two more variations;</div><div><br></div><div>- OptionalModuleLoader.patch -- what you suggested above. Needs a flag and dynamic allocation and lots of point changes to go from ref to pointer.</div><div>
- ImpureModuleLoader.patch -- what I suggested later; have ModuleLoader not be a protocol class, but give loadModule a default implementation. I don't know if it's important for ModuleLoader to be pure.</div><div>
<br></div><div>For what it's worth,</div><div>- Kim</div></div>