Depends on D697
Diff Detail
Diff Detail
- Repository
- rMOZILLACENTRAL mozilla-central
- Branch
- HEAD
Event Timeline
Comment Actions
Code analysis found 6 defects in this patch:
- 6 defects found by clang-tidy
You can run this analysis locally with:
- ./mach static-analysis check path/to/file.cpp (C/C++)
If you see a problem in this automated review, please report it here: https://bit.ly/2tb8Qk3
| js/src/jsapi.cpp | ||
|---|---|---|
| 237 | Warning: Use '= default' to define a trivial default constructor [clang-tidy: modernize-use-equals-default] B() {}
^ ~~
= default; | |
| 238 | Warning: Use '= default' to define a trivial copy constructor [clang-tidy: modernize-use-equals-default] B(const B&) {}
^ ~~
= default; | |
| 239 | Warning: Move constructors should be marked noexcept [clang-tidy: performance-noexcept-move-constructor] B(B &&) {}
^ | |
| 244 | Warning: Use '= default' to define a trivial copy constructor [clang-tidy: modernize-use-equals-default] D(const D &RHS) : B(RHS) {}
^ | |
| 245 | Warning: Move constructor initializes base class by calling a copy constructor [clang-tidy: performance-move-constructor-init] D(D &&RHS) : B(RHS) {}
^ | |
| 245 | Warning: Move constructors should be marked noexcept [clang-tidy: performance-noexcept-move-constructor] D(D &&RHS) : B(RHS) {}
^ | |