- User Since
- Jul 27 2016, 4:23 AM (255 w, 4 d)
Thu, Jun 3
You have, with pragma gcc error, look at hb.hh. The problem is that clang has an error dealing with cascading pragma when setting different levels.
Tue, Jun 1
I think there is a false positive with this @george.burgess.iv:
In [this)(https://searchfox.org/mozilla-central/source/mozglue/baseprofiler/core/platform-linux-android.cpp#222-227) we have the warning triggered:
mozglue/baseprofiler/core/platform-linux-android.cpp:216:9: error: variable 'r' set but not used [-Werror,-Wunused-but-set-variable]
May 6 2021
May 3 2021
I'm seeing here something very strange, if the user provided copy assignment operator is provided but is = delete and the copy constructor is default this warning will still trigger. Is this something expected?
I'm referring at this issue from the mozilla code-base.
Apr 28 2021
Thank you for this! Also I think it’s very well that you want to implement this warning in clang, with a little bit of more polish it will be up for merger!
Please revert this. This change hasn’t been tested nor reviewed enough. Also I don’t remember seeing this proposed on cfe dev mailing list.
Apr 27 2021
I think this added a regression, where you have this context:
Apr 16 2021
Apr 12 2021
Add -use-color argument to make it sane with clang-tidy --use-color argument.
Feb 12 2021
This has landed and trunk is already fixed.
Feb 8 2021
Landed in 11.1 https://github.com/llvm/llvm-project/commit/1fdec59bffc11ae37eb51a1b9869f0696bfd5312#diff-2fa23ad0cf1839955ddaf4a0d78a9d9b5fd9b88933f82f6433035916a2655c6c.
Patch needs to be rebased for trunk now.
Feb 4 2021
The failure is wrong.
Feb 3 2021
rebased on 11.1rc2
Jan 30 2021
@tstellar what do you suggest to do here?
Jan 29 2021
Update for plularization of variables name.
Update with format
And there are more issues like: https://github.com/llvm/llvm-project/blob/llvmorg-11.0.1/lldb/source/Plugins/Platform/MacOSX/PlatformDarwinKernel.cpp#L733. old_module_sp_ptr is nowhere declared.
Shouldn't this also update PlatformiOSSimulator.h?
Dec 18 2020
Oct 9 2020
Sep 15 2020
Has been pushed to the 11.x branch https://github.com/llvm/llvm-project/commit/1596c2dfd548b21cf33ad3353882ac465d78c1bb
Jul 2 2020
@njames93 wdyt if we add another parameter to distinguish if we want to use regex or not, and if not we escape the paths?
Also thank you so much for catching this up!
Jun 29 2020
Jun 16 2020
May 19 2020
Let's first see we don't break anything on mozilla.
May 15 2020
@MyDeveloperDay thanks for the patch, I'm gonna run it agains mozilla to see if there is any fallout.
May 12 2020
Sorry, totally forgot about this, thank you!
May 3 2020
As per what @sammccall said.
Apr 28 2020
Just to clarify something here, for me the patch looks good but until I will accept the revision I want to test it on the mozilla codebase.
Apr 27 2020
I've cherry-picked D76850 to 10.x and see if this fixes the issue.
I think we are on the right track with this but what if we are dealing with tok::amp in a context similar to operator const FallibleTArray<E>&(); I can speculate that the outcome will be operator const FallibleTArray<E> &(); which is wrong.
Apr 11 2020
Add release notes.
Apr 10 2020
Feb 24 2020
Feb 5 2020
Yes I have commit access, pushing it directly to 10.x then.
Adding extra comments to the release patch.
Jan 9 2020
Jan 8 2020
Will push shortly an update.
I think the best approach here is to remove the tests that don’t have template instantiation.
Added more tesst for templa functions that are not instantiated.
Jan 7 2020
Updated test with match without compound statement.
Updated the test case to better reflect the changes.
Sep 5 2019
I think the auto usage improves and simplifies the code, however, as I would really like to see this code landed, I would be ok to help Stephen to finish this work and replace auto by the "old" way. Do you think we can have this approved if we make the changes mentioned earlier?