Yes! That patch works great for me!<div><br></div><div>Thanks,</div><div>Nicolas<br><br><div class="gmail_quote">On Tue, Dec 22, 2009 at 8:52 PM, Jeffrey Yasskin <span dir="ltr"><<a href="mailto:jyasskin@google.com">jyasskin@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">On Mon, Dec 21, 2009 at 9:38 PM, nicolas geoffray<br>
<div class="im"><<a href="mailto:nicolas.geoffray@gmail.com">nicolas.geoffray@gmail.com</a>> wrote:<br>
</div><div class="im">> Hi Jeffrey,<br>
><br>
>><br>
>> Ah, I see now. You have your own ModuleProvider, and it needs to know<br>
>> precisely when the function is actually needed, right? Yeah, it sounds<br>
>> like I need to roll back that part of this fix until PR5737 is fixed.<br>
>><br>
><br>
> Yes, thanks!<br>
<br>
</div>How's the attached patch look? I discovered that one of the<br>
isDeclaration()s was untested, so I added a test for it.<br>
</blockquote></div><br></div>