<div dir="ltr"><br><br><div class="gmail_quote">On Mon, Apr 20, 2015 at 4:35 PM Vladimir Voskresensky <<a href="mailto:vladimir.voskresensky@oracle.com">vladimir.voskresensky@oracle.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div bgcolor="#FFFFFF" text="#000000">
Hello Nikola,<br>
<br>
I've created <a href="http://reviews.llvm.org/D9109" target="_blank">http://reviews.llvm.org/D9109</a>, but for unknown reason
as soon as focus lost "reviewer" or "Subscribers" fields it's
content is erased, so I can not add you and cfe list there.</div></blockquote><div><br></div><div>What browser are you using?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div bgcolor="#FFFFFF" text="#000000"> Could
you update <a href="http://reviews.llvm.org/D9109" target="_blank">http://reviews.llvm.org/D9109</a> appropriately.<br>
<br>
Thank you!</div><div bgcolor="#FFFFFF" text="#000000"><br>
Vladimir.</div><div bgcolor="#FFFFFF" text="#000000"><br>
<br>
<div>On 04/18/2015 04:24 AM, Nikola
Smiljanic wrote:<br>
</div>
<blockquote type="cite">
<div dir="ltr">Sorry for not telling you earlier, can you please
upload this to Phabricator and CC me?</div>
<div class="gmail_extra"><br>
<div class="gmail_quote">On Sat, Apr 18, 2015 at 4:43 AM,
Vladimir Voskresensky - Oracle <span dir="ltr"><<a href="mailto:vladimir.voskresensky@oracle.com" target="_blank">vladimir.voskresensky@oracle.com</a>></span>
wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div text="#000000" bgcolor="#FFFFFF"> <tt>Hello Nikola,<br>
<br>
Please, review and apply attached patch.<br>
<br>
Thanks!<span><font color="#888888"><br>
Vladimir.<br>
<br>
</font></span></tt>
<div>
<div>
<div>On 04/14/15 11:14 AM, Nikola Smiljanic wrote:<br>
</div>
<blockquote type="cite">
<div dir="ltr">Looks that way at first glance, feel
free to submit a patch as there seem to be quite a
few places that need updating.</div>
<div class="gmail_extra"><br>
<div class="gmail_quote">On Sat, Apr 4, 2015 at
1:15 AM, Vladimir Voskresensky - Oracle <span dir="ltr"><<a href="mailto:vladimir.voskresensky@oracle.com" target="_blank">vladimir.voskresensky@oracle.com</a>></span>
wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br>
<br>
It looks like the name and documentation of
the first parameter is incorrect<br>
virtual void FileSkipped(const FileEntry
&ParentFile,<br>
const Token
&FilenameTok,<br>
SrcMgr::CharacteristicKind FileType) {<br>
}<br>
<br>
passed ParentFile is in fact SkippedFile.<br>
<br>
The code from
Preprocessor::HandleIncludeDirective which
calls FileSkipped:<br>
<br>
// Ask HeaderInfo if we should enter this
#include file. If not, #including<br>
// this file will have no effect.<br>
if (!HeaderInfo.ShouldEnterIncludeFile(File,
isImport)) {<br>
if (Callbacks)<br>
Callbacks->FileSkipped(*File,
FilenameTok, FileCharacter);<br>
return;<br>
}<br>
<br>
<br>
Thanks,<br>
Vladimir.<br>
<br>
_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@cs.uiuc.edu" target="_blank">cfe-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>
</blockquote>
</div>
<br>
</div>
</blockquote>
<br>
</div>
</div>
</div>
</blockquote>
</div>
<br>
</div>
<br>
<fieldset></fieldset>
<br>
<pre>_______________________________________________
cfe-dev mailing list
<a href="mailto:cfe-dev@cs.uiuc.edu" target="_blank">cfe-dev@cs.uiuc.edu</a>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a>
</pre>
</blockquote>
<br>
</div>
_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@cs.uiuc.edu" target="_blank">cfe-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>
</blockquote></div></div>