Details
- Reviewers
- None
- Bugzilla Bug ID
- 1682030
Diff Detail
- Repository
- rMOZILLACENTRAL mozilla-central
- Branch
- default
- Lint
No Lint Coverage Severity Location Code Message Error dom/plugins/base/nsNPAPIPluginInstance.h:86 clang-diagnostic-error Build Error Error dom/plugins/base/nsNPAPIPluginInstance.h:116 clang-diagnostic-error Build Error Error dom/plugins/base/nsNPAPIPluginInstance.h:117 clang-diagnostic-error Build Error Error dom/plugins/base/nsNPAPIPluginStreamListener.h:34 clang-diagnostic-error Build Error Error dom/plugins/base/nsNPAPIPluginStreamListener.h:64 clang-diagnostic-error Build Error Error dom/plugins/base/nsNPAPIPluginStreamListener.h:65 clang-diagnostic-error Build Error Error dom/plugins/base/nsNPAPIPluginStreamListener.h:85 clang-diagnostic-error Build Error Error dom/plugins/base/nsPluginHost.h:29 clang-diagnostic-error Build Error Error dom/plugins/ipc/PluginInstanceChild.cpp:300 clang-diagnostic-error Build Error Error dom/plugins/ipc/PluginInstanceChild.cpp:639 clang-diagnostic-error Build Error Error dom/plugins/ipc/PluginModuleParent.h:14 clang-diagnostic-error Build Error Error toolkit/xre/nsEmbedFunctions.cpp:528 clang-diagnostic-error Build Error Warning dom/plugins/ipc/PluginModuleParent.cpp:1 no-coverage code coverage analysis Warning dom/plugins/ipc/PluginProcessChild.cpp:1 no-coverage code coverage analysis Warning dom/plugins/ipc/PluginProcessParent.cpp:1 no-coverage code coverage analysis - Unit
No Test Coverage
Event Timeline
Code analysis found 15 defects in the diff 378009:
- 12 build errors found by clang-tidy
- 3 defects found by code coverage analysis
You can run this analysis locally with:
- ./mach static-analysis check --outgoing (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 -p0).
The analysis task source-test-coverity-coverity failed, but we could not detect any issue.
Please check this task manually.
The analysis task source-test-mozlint-rejected-words failed, but we could not detect any issue.
Please check this task manually.
In our previous code coverage analysis run, we found some files which had no coverage and are being modified in this patch:
Should they have tests, or are they dead code?
- You can file a bug blocking Bug 1415824 for untested files that should be tested.
- You can file a bug blocking Bug 1415819 for untested files that should be removed.
If you see a problem in this automated review, please report it here.
You can view these defects on the code-review frontend and on Treeherder.
Code analysis found 15 defects in the diff 378147:
- 12 build errors found by clang-tidy
- 3 defects found by code coverage analysis
You can run this analysis locally with:
- ./mach static-analysis check --outgoing (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 -p0).
The analysis task source-test-coverity-coverity failed, but we could not detect any issue.
Please check this task manually.
The analysis task source-test-mozlint-rejected-words failed, but we could not detect any issue.
Please check this task manually.
In our previous code coverage analysis run, we found some files which had no coverage and are being modified in this patch:
Should they have tests, or are they dead code?
- You can file a bug blocking Bug 1415824 for untested files that should be tested.
- You can file a bug blocking Bug 1415819 for untested files that should be removed.
If you see a problem in this automated review, please report it here.
You can view these defects on the code-review frontend and on Treeherder.