- User Since
- Mar 19 2021, 3:29 PM (79 w, 3 d)
Tue, Sep 20
Tue, Sep 13
Tue, Sep 6
Mon, Aug 29
I feel like Phabricator is not sending notifications for this patch like it usually does (I'm not getting any emails).
I'll create a new, identical patch tomorrow if there's still no activity.
Aug 23 2022
Aug 16 2022
Jul 1 2022
I do not have commit access for the LLVM repository, could one of you commit this for me (Nicolas van Kempen <firstname.lastname@example.org>)?
Jun 30 2022
Did some digging, looks like this was added in rG7f92a1a84b96, most likely to also fix that CI build error with misc-misleading-identifier.
Jun 29 2022
Ping, adding one more person who has history changing this script.
Jun 20 2022
Jun 16 2022
Looks like registerPPCallbacks / Result.Context->getSourceManager() gives me the information I need. Thank you @gribozavr2 for your help!
Jun 15 2022
Thanks! Looks like registerPPCallbacks might be what I am looking for, I was not aware of it. I will mark this as planning changes temporarily, then abandon if solved.
Looks like this causes the misc-misleading-identifier test to fail on the BuildKite Windows setup because it uses cp1252 and not utf-8. I think this is why the encode was added in the first place. This workaround should hopefully fix this, it will display unicode characters as '?' when stdout does not support utf-8.
Jun 14 2022
Jun 2 2022
May 23 2022
Fix formatting issues
- Fixed a minor bug which occasionaly caused false negatives.
- Cleared up fix/hint generation and another nit following comments.
- Re-organized and added a couple tests.
- Made a note of this extension in documentation.
May 6 2021
Fix some style comments
Apr 28 2021
Thank you for the feedback, I appreciate it! Let me know if there is any other I missed, AFAIK all auto uses introduced by me should be good now.
Explicitely specify one more type
Fix a couple variable names and explicitely specify a couple types
As you can see, this will fail some tests as there is one corner case which is tricky to handle:
std::vector<std::tuple<std::tuple<int, int>>> vec; // This is valid and should not be changed. vec.emplace_back(std::make_tuple(13, 31));