Fixed misleading parameter name for PPCallbacks::FileSkipped

[voskresensky.vladimir]

Summary

The first parameter of PPCallbacks::FileSkipped callback is SkippedFile, not the ParentFile according to the call at Preprocessor::HandleIncludeDirective

[nikola]

Your original patch had changes in the clang-extra repo as well but those are missing in this review, was that intentional?

include/clang/Lex/PPCallbacks.h

59 ↗

(On Diff #24021)

You can probably remove ParentFile from this sentence.

[voskresensky.vladimir]

Sorry for the long delay! I've attached changes from 'clang-extra'

[nikola]

No worries. This looks good, just fix the ParentFile reference I mentioned.

[voskresensky.vladimir]

updated comments

[nikola]

LGTM, would you like me to commit this if you don't have access?

[voskresensky.vladimir]

Nikola, I don't have write access, so I'd appreciate your help with commit.

Btw, the updated diff was again only about clang repository, not extra. I used "update current" instead of "create new diff" was that correct? I do not see extra files in the list now, they are only in history of http://reviews.llvm.org/differential/diff/25416. What was the right way to use phabricator for such use case?

Thanks!
Vladimir.

[nikola]

I have no idea to be honest, the fact that we have only one svn repository but checkout different parts separately makes things confusing. And since I'm using git it's even more complicated ;)

Thanks for this, committed r237115 and r237116

pp-trace/PPCallbacksTracker.h

                    clang::PPCallbacks::FileChangeReason Reason,
                    clang::SrcMgr::CharacteristicKind FileType,
                    clang::FileID PrevFID = clang::FileID()) override;
-  void FileSkipped(const clang::FileEntry &ParentFile,
+  void FileSkipped(const clang::FileEntry &SkippedFile,
                    const clang::Token &FilenameTok,
                    clang::SrcMgr::CharacteristicKind FileType) override;
   bool FileNotFound(llvm::StringRef FileName,

pp-trace/PPCallbacksTracker.cpp

 // Callback invoked whenever a source file is skipped as the result
 // of header guard optimization.
 void
-PPCallbacksTracker::FileSkipped(const clang::FileEntry &ParentFile,
+PPCallbacksTracker::FileSkipped(const clang::FileEntry &SkippedFile,
                                 const clang::Token &FilenameTok,
                                 clang::SrcMgr::CharacteristicKind FileType) {
   beginCallback("FileSkipped");
-  appendArgument("ParentFile", &ParentFile);
+  appendArgument("ParentFile", &SkippedFile);
   appendArgument("FilenameTok", FilenameTok);
   appendArgument("FileType", FileType, CharacteristicKindStrings);
 }