<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::<u></u>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.<u></u>ShouldEnterIncludeFile(File, isImport)) {<br>
    if (Callbacks)<br>
      Callbacks->FileSkipped(*File, FilenameTok, FileCharacter);<br>
    return;<br>
  }<br>
<br>
<br>
Thanks,<br>
Vladimir.<br>
<br>
______________________________<u></u>_________________<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/<u></u>mailman/listinfo/cfe-dev</a><br>
</blockquote></div><br></div>