
<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div>Could we change the Preprocessor to accept an optional pointer to a ModuleLoader ?</div><div><br></div><div>This will eliminate the whole "VoidModuleLoader" business and the hack of having the ASTUnit "implement" a ModuleLoader just so we have something to pass to the Preprocessor when loading an AST file.</div><div><br></div><div>-Argyrios</div><div><br></div><div>On Nov 5, 2012, at 11:22 AM, Kim Gräsman <<a href="mailto:kim.grasman@gmail.com">kim.grasman@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">Hi Chandler,<div><br></div><div>Sure, I can't add much more to that end, but I agree that this is probably better off in a library shared between unit tests.</div><div><br></div><div>- Kim<br><br><div class="gmail_quote">

On Mon, Nov 5, 2012 at 12:39 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

I like the direction of this, but I think it should go into a<br>

unittest-specific library.<br>

<br>

We don't currently have any such library infrastructure, but honestly,<br>

I think we need it and we should get it in place sooner rather than<br>

later.<br>

<br>

Adding Manuel as he may have ideas here.<br>

<div><div class="h5"><br>

On Sun, Nov 4, 2012 at 11:27 AM, Kim Gräsman <<a href="mailto:kim.grasman@gmail.com">kim.grasman@gmail.com</a>> wrote:<br>

> Hi committers,<br>

><br>

> I contributed a small patch a few weeks ago, adding unittests for<br>

> PPCallbacks::InclusionDirective.<br>

><br>

> I was bugged by the proliferation of stub ModuleLoader implementations but<br>

> didn't want to do anything about it in that context.<br>

><br>

> Here's a patch to move VoidModulerLoader into ModuleLoader.h and remove all<br>

> redundant implementations.<br>

><br>

> Could someone review and commit if this is acceptable?<br>

><br>

> Thanks,<br>

> - Kim<br>

><br>

</div></div>> _______________________________________________<br>

> cfe-commits mailing list<br>

> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>

> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>

><br>

</blockquote></div><br></div>

_______________________________________________<br>cfe-commits mailing list<br><a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits<br></blockquote></div><br></body></html>