Please use GitHub pull requests for new patches. Avoid migrating existing patches. Phabricator shutdown timeline
Feb 8 2023
Jan 31 2023
Nevermind, this was fixed as I was looking at it. LGTM.
This is becoming repetitive, but I guess that's the nature of those things. Maybe something like this would help clean it up, not sure if any better for right now.
Dec 1 2022
Nov 17 2022
@njames93 If you have a minute, could you please take a second look at this? Let me know what's needed to get this change landed.
Oct 11 2022
Sep 30 2022
Thank you for the feedback!
Rename OnlyWarnInLoops to WarnOutsideLoops, cover missed cxxForRangeStmt
Sep 27 2022
Sep 20 2022
Sep 13 2022
Sep 6 2022
Aug 29 2022
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));