Depends on D895
Details
Diff Detail
- Repository
- rMOZILLACENTRAL mozilla-central
Event Timeline
Code analysis found 3 defects in this patch:
- 2 defects found by clang-format
- 1 defect found by clang-tidy
You can run this analysis locally with:
- ./mach clang-format -p path/to/file.cpp (C/C++)
- ./mach static-analysis check path/to/file.cpp (C/C++)
For your convenience, here is a patch that fixes all the clang-format defects (use it in your repository with hg import or git apply).
If you see a problem in this automated review, please report it here.
| js/src/vm/Interpreter.cpp | ||
|---|---|---|
| 1635 | Warning: Macro argument should be enclosed in parentheses [clang-tidy: bugprone-macro-parentheses] obj = ToObjectFromStack((cx), (val)); \ ^~~ () | |
Code analysis found 3 defects in this patch:
- 2 defects found by clang-format
- 1 defect found by clang-tidy
You can run this analysis locally with:
- ./mach clang-format -p path/to/file.cpp (C/C++)
- ./mach static-analysis check path/to/file.cpp (C/C++)
For your convenience, here is a patch that fixes all the clang-format defects (use it in your repository with hg import or git apply).
If you see a problem in this automated review, please report it here.
| js/src/vm/Interpreter.cpp | ||
|---|---|---|
| 1635 | Warning: Macro argument should be enclosed in parentheses [clang-tidy: bugprone-macro-parentheses] obj = ToObjectFromStack((cx), (val)); \ ^~~ () | |