Page MenuHomePhabricator Dev

Bug 1630534 - Code review integration test
Needs ReviewPublic

Authored by andi on May 1 2020, 7:03 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Oct 11, 10:10 AM
Unknown Object (File)
Fri, Oct 9, 1:01 PM
Unknown Object (File)
Fri, Oct 9, 1:00 PM
Unknown Object (File)
Fri, Oct 9, 1:00 PM
Unknown Object (File)
Fri, Oct 9, 12:58 PM
Unknown Object (File)
Tue, Oct 6, 10:00 AM
Unknown Object (File)
Tue, Oct 6, 10:00 AM
Unknown Object (File)
Tue, Oct 6, 10:00 AM
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Bugzilla Bug ID
1630534

Diff Detail

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes

Code analysis found 3 defects in the diff 5061:

  • 1 build error and 1 defect found by clang-tidy
  • 1 defect found by Coverity

You can run this analysis locally with:

  • ./mach static-analysis check --outgoing (C/C++)

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 2 defects in the diff 5067:

  • 1 build error and 1 defect found by clang-tidy

You can run this analysis locally with:

  • ./mach static-analysis check --outgoing (C/C++)

The analysis task source-test-coverity-coverity failed, but we could not detect any issue.
Please check this task manually.

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 2 defects in the diff 5068:

  • 1 build error and 1 defect found by clang-tidy

You can run this analysis locally with:

  • ./mach static-analysis check --outgoing (C/C++)

The analysis task source-test-coverity-coverity failed, but we could not detect any issue.
Please check this task manually.

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 7 defects in the diff 5084:

  • 1 build error and 2 defects found by clang-tidy
  • 1 defect found by clang-format
  • 3 defects found by Coverity

You can run this analysis locally with:

  • ./mach clang-format -p dom/animation/Animation.cpp
  • ./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).

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.

andi changed the repository for this revision from rMC m-c to Restricted Repository.
andi changed the repository for this revision from Restricted Repository to rMC m-c.
andi changed the repository for this revision from rMC m-c to rMOZILLACENTRAL mozilla-central.

Code analysis found 7 defects in the diff 5090:

  • 1 build error and 2 defects found by clang-tidy
  • 1 defect found by clang-format
  • 3 defects found by Coverity

You can run this analysis locally with:

  • ./mach clang-format -p dom/animation/Animation.cpp
  • ./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).

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 7 defects in the diff 5091:

  • 1 build error and 2 defects found by clang-tidy
  • 1 defect found by clang-format
  • 3 defects found by Coverity

You can run this analysis locally with:

  • ./mach clang-format -p dom/animation/Animation.cpp
  • ./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).

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 7 defects in the diff 5093:

  • 1 build error and 2 defects found by clang-tidy
  • 1 defect found by clang-format
  • 3 defects found by Coverity

You can run this analysis locally with:

  • ./mach clang-format -p dom/animation/Animation.cpp
  • ./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).

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 7 defects in the diff 5092:

  • 1 build error and 2 defects found by clang-tidy
  • 1 defect found by clang-format
  • 3 defects found by Coverity

You can run this analysis locally with:

  • ./mach clang-format -p dom/animation/Animation.cpp
  • ./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).

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 7 defects in the diff 5094:

  • 1 build error and 2 defects found by clang-tidy
  • 1 defect found by clang-format
  • 3 defects found by Coverity

You can run this analysis locally with:

  • ./mach clang-format -p dom/animation/Animation.cpp
  • ./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).

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 7 defects in the diff 5095:

  • 1 build error and 2 defects found by clang-tidy
  • 1 defect found by clang-format
  • 3 defects found by Coverity

You can run this analysis locally with:

  • ./mach clang-format -p dom/animation/Animation.cpp
  • ./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).

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.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

Code analysis found 7 defects in the diff 5096:

  • 1 build error and 2 defects found by clang-tidy
  • 1 defect found by clang-format
  • 3 defects found by Coverity

You can run this analysis locally with:

  • ./mach clang-format -p dom/animation/Animation.cpp
  • ./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).

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.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

Code analysis found 7 defects in the diff 5097:

  • 1 build error and 2 defects found by clang-tidy
  • 1 defect found by clang-format
  • 3 defects found by Coverity

You can run this analysis locally with:

  • ./mach clang-format -p dom/animation/Animation.cpp
  • ./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).

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 7 defects in the diff 5098:

  • 1 build error and 2 defects found by clang-tidy
  • 1 defect found by clang-format
  • 3 defects found by Coverity

You can run this analysis locally with:

  • ./mach clang-format -p dom/animation/Animation.cpp
  • ./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).

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.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

Code analysis found 7 defects in the diff 5099:

  • 1 build error and 2 defects found by clang-tidy
  • 1 defect found by clang-format
  • 3 defects found by Coverity

You can run this analysis locally with:

  • ./mach clang-format -p dom/animation/Animation.cpp
  • ./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).

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 7 defects in the diff 5100:

  • 1 build error and 2 defects found by clang-tidy
  • 1 defect found by clang-format
  • 3 defects found by Coverity

You can run this analysis locally with:

  • ./mach clang-format -p dom/animation/Animation.cpp
  • ./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).

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.

andi retitled this revision from Bug XXYYZZ - Code review integration test to Bug 1630534 - Code review integration test.
andi added Bugzilla Bug ID 1630534.

Code analysis found 7 defects in the diff 5101:

  • 1 build error and 2 defects found by clang-tidy
  • 1 defect found by clang-format
  • 3 defects found by Coverity

You can run this analysis locally with:

  • ./mach clang-format -p dom/animation/Animation.cpp
  • ./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).

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 7 defects in the diff 5102:

  • 1 build error and 2 defects found by clang-tidy
  • 1 defect found by clang-format
  • 3 defects found by Coverity

You can run this analysis locally with:

  • ./mach clang-format -p dom/animation/Animation.cpp
  • ./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).

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 7 defects in the diff 5103:

  • 1 build error and 2 defects found by clang-tidy
  • 1 defect found by clang-format
  • 3 defects found by Coverity

You can run this analysis locally with:

  • ./mach clang-format -p dom/animation/Animation.cpp
  • ./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).

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.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

The analysis task source-test-clang-tidy failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.