Changes

Jump to: navigation, search

Static Analysis

1,259 bytes removed, 18:19, 10 September 2020
content moved
== Current Status == * '''Custom static analyses''': We have a clang plugin with a number of Gecko-specific checks. There's terse documentation on the attributes we use to drive some of the checks [https://dxr.mozilla.org/mozilla-central/source/mfbt/Attributes.h#341 Content moved here]. Some checks are just good hygiene (e.g. MOZ_IMPLICIT), some checks exist to help you do the right thing (e.g. MOZ_MUST_OVERRIDE, MOZ_RAII, MOZ_MUST_USE), and some checks exist to prevent security bugs (e.g. MOZ_NON_MEMMOVABLE and related attributes). The checker currently runs on every push we do, on Windows, Mac, and Linux. * '''clang upstream analyses''': For every patch, at review phase, we run a bunch of [https://searchfox.org/mozilla-central/source/tools/clang-tidy/config.yaml clang analyzer and clang tidy] checkers. * '''[https://scan.coverity.com/projects/firefox Coverity]''': Executed at review phase for every patch. * We are also running a bunch of linters: https://firefox-source-docs.mozilla.org/toolscode-quality/lint/indexstatic-analysis.html * '''Compiler warnings''': all of our compilers have a number of warnings. We try to turn on as many as we can, and make warnings on most Mozilla code fatal, i.e. your build will fail if the compiler warns. We generally turn off fatal warnings for third-party code, and sometimes attempt to get fixes for the warnings pushed upstream.
Confirm
709
edits

Navigation menu