Граф коммитов

118 Коммитов

Автор SHA1 Сообщение Дата
Mike Hommey 79fc010202 Bug 1801058 - Update builders to clang 15.0.5. r=firefox-build-system-reviewers,sergesanspaille
Differential Revision: https://phabricator.services.mozilla.com/D162273
2022-11-17 19:57:30 +00:00
Mike Hommey cd2c134e91 Bug 1799690 - Update clang-tidy version after clang update. r=firefox-build-system-reviewers,andi
Differential Revision: https://phabricator.services.mozilla.com/D161631
2022-11-09 20:41:23 +00:00
Mike Hommey bee90b0f95 Bug 1784202 - Update builders to clang 15. r=firefox-build-system-reviewers,andi
We keep clang 14 for webrender-wrench for now, because of bug 1789346.

Differential Revision: https://phabricator.services.mozilla.com/D155532
2022-11-01 23:46:33 +00:00
Sandor Molnar 55c9f970f6 Backed out changeset 59151d6a9b0b (bug 1784202) for causing crashes @strlen + 0x30 2022-10-26 14:59:24 +03:00
Mike Hommey cfce5ea6fe Bug 1784202 - Update builders to clang 15. r=firefox-build-system-reviewers,andi
We keep clang 14 for webrender-wrench for now, because of bug 1789346.

Differential Revision: https://phabricator.services.mozilla.com/D155532
2022-10-25 07:28:24 +00:00
Marian-Vasile Laza dca276a235 Backed out changeset 1efa06d7902a (bug 1784202) for causing mochitest failures. CLOSED TREE 2022-09-22 04:44:50 +03:00
Mike Hommey 70cdc9a963 Bug 1784202 - Update builders to clang 15. r=firefox-build-system-reviewers,andi
We keep clang 14 for webrender-wrench for now, because of bug 1789346.

Differential Revision: https://phabricator.services.mozilla.com/D155532
2022-09-21 20:11:34 +00:00
Butkovits Atila 86987f69cb Backed out changeset 9d40b66a9202 (bug 1784202) for causing build bustages. 2022-09-21 15:05:08 +03:00
Mike Hommey ba0be83d9b Bug 1784202 - Update builders to clang 15. r=firefox-build-system-reviewers,andi
We keep clang 14 for webrender-wrench for now, because of bug 1789346.

Differential Revision: https://phabricator.services.mozilla.com/D155532
2022-09-21 09:17:18 +00:00
Mike Hommey a68bf5548f Bug 1774515 - Upgrade clang to 14.0.5. r=firefox-build-system-reviewers,andi
Differential Revision: https://phabricator.services.mozilla.com/D149467
2022-06-16 04:47:44 +00:00
Mike Hommey 35dbe9c4e7 Bug 1771483 - Bump clang-tidy version
Differential Revision: https://phabricator.services.mozilla.com/D147549
2022-05-27 21:21:53 +00:00
Mike Hommey e6c12dc71e Bug 1767372 - Upgrade clang to 14.0.3. r=firefox-build-system-reviewers,andi
Differential Revision: https://phabricator.services.mozilla.com/D145294
2022-05-05 07:16:46 +00:00
Mike Hommey 51ac12e4a3 Bug 1764509 - Update builders to clang 14.0.1. r=firefox-build-system-reviewers,andi
Differential Revision: https://phabricator.services.mozilla.com/D143570
2022-04-22 01:52:43 +00:00
Mike Hommey 548aade9e4 Bug 1758780 - Update builders to clang 14. r=firefox-build-system-reviewers,mhentges,andi
Differential Revision: https://phabricator.services.mozilla.com/D143175
2022-04-21 04:36:24 +00:00
Andi-Bogdan Postelnicu 40ca2d63f3 Bug 1744446 - Upgrade llvm toolchain to 13.0.1 r=firefox-build-system-reviewers,mhentges
Differential Revision: https://phabricator.services.mozilla.com/D132870
2022-02-03 15:18:33 +00:00
Andi-Bogdan Postelnicu 0d12e234b2 Bug 1731582 - Move to clang-13 r=firefox-build-system-reviewers,mhentges,decoder
Join work with glandium.

Differential Revision: https://phabricator.services.mozilla.com/D126208
2021-10-20 12:03:20 +00:00
Andi-Bogdan Postelnicu 90ec98c524 Bug 1719812 - Upgrade llvm toolchain to 12.0.1. r=firefox-build-system-reviewers,mhentges
Differential Revision: https://phabricator.services.mozilla.com/D119493
2021-07-12 13:44:32 +00:00
Andi-Bogdan Postelnicu 6666ac0af8 Bug 1696180 - Update Clang based static analysis to clang 12. r=firefox-build-system-reviewers,mhentges
Differential Revision: https://phabricator.services.mozilla.com/D113330
2021-04-28 09:56:13 +00:00
Jeff Gilbert 85bbcbfa5e Bug 1602143 - Use ShortStatementLines=1 for readability-braces-around-statements. r=andi
This will reduce the false-positive rate that sometimes causes devs to
ignore other important warnings because they expect to see these
warnings because of module/legacy code styles.

```
if (foo) return; // no longer warns
if (bar)
  return; // still warns
```
clang-format will sometimes turn long-line versions of the former into
the latter, and this warning will pick those as readability warnings.
NB: This is Chromium style for same-line conditional statements.

Another test result json has its EOL-newline removed too, since that's
what the script does and I'm just committing its results.

Differential Revision: https://phabricator.services.mozilla.com/D105497
2021-02-18 08:26:11 +00:00
David Major 180d35da9f Bug 1685335 - Update to clang 11.0.1 r=firefox-build-system-reviewers,static-analysis-reviewers,andi,mhentges,sylvestre
In addition to the usual dot-release type of fixes, this also lets us drop a good amount of code that we had patched into our clang 11.

Differential Revision: https://phabricator.services.mozilla.com/D100959
2021-01-07 13:53:53 +00:00
Kartik Gautam 7ae6aea145 Bug 1684173 - Add newline character at end of files when missing r=sylvestre,geckoview-reviewers,agi
Differential Revision: https://phabricator.services.mozilla.com/D100484
2021-01-07 08:53:08 +00:00
Cosmin Sabou 2978aa00a3 Backed out changeset dbed1cdf588f (bug 1684173) for mochitest plain and devtools failures. a=backout DONTBUILD 2020-12-28 00:43:51 +02:00
Kartik Gautam 775cdec032 Bug 1684173 - Add newline character at end of files when missing r=sylvestre
Depends on D100443

Differential Revision: https://phabricator.services.mozilla.com/D100484
2020-12-27 11:43:41 +00:00
Sylvestre Ledru 20512e8f40 Bug 1674633 - clang-tidy: disable WarnOnConditionVariables for readability-else-after-return r=static-analysis-reviewers,andi DONTBUILD
Differential Revision: https://phabricator.services.mozilla.com/D100085
2020-12-18 08:29:08 +00:00
Kartik Gautam f7ffcd09fb Bug 1679758 - Remove trailing empty lines r=sylvestre,marionette-reviewers,jgraham
Differential Revision: https://phabricator.services.mozilla.com/D99595
2020-12-15 10:34:54 +00:00
Dorel Luca 7320ae982a Backed out changeset f3aaf04fce3b (bug 1679758) for Devtool failures in browser_styleeditor_syncAddProperty.js. CLOSED TREE 2020-12-13 16:38:21 +02:00
Kartik Gautam caf549c200 Bug 1679758 - Remove trailing empty lines r=sylvestre
Differential Revision: https://phabricator.services.mozilla.com/D99595
2020-12-13 13:28:30 +00:00
Andi-Bogdan Postelnicu e9fbba867c Bug 1666550 - Update `clang-tools` to `clang-11`. r=dmajor
Differential Revision: https://phabricator.services.mozilla.com/D97137
2020-11-17 15:04:14 +00:00
Andi-Bogdan Postelnicu 8731a55350 Bug 1612274 - Add checker `readability-qualified-auto` for `clang-tidy` static-analysis. r=sylvestre
Differential Revision: https://phabricator.services.mozilla.com/D70481

--HG--
extra : moz-landing-system : lando
2020-04-10 07:06:09 +00:00
Andi-Bogdan Postelnicu 56a9802f89 Bug 1625884 - move `clang-tidy` and `clang-format` to `clang-10`. r=nalexander
Differential Revision: https://phabricator.services.mozilla.com/D70065

--HG--
extra : moz-landing-system : lando
2020-04-09 14:16:44 +00:00
Cosmin Sabou 60934f9072 Backed out 3 changesets (bug 1625884) as requested by ANdi for causing build bustages. a=backout
Backed out changeset a35cfda6e271 (bug 1625884)
Backed out changeset 412c8c401196 (bug 1625884)
Backed out changeset 927ff250bc4b (bug 1625884)
2020-04-09 16:14:06 +03:00
Andi-Bogdan Postelnicu ffe0a6a754 Bug 1625884 - move `clang-tidy` and `clang-format` to `clang-10`. r=nalexander
Differential Revision: https://phabricator.services.mozilla.com/D70065

--HG--
extra : moz-landing-system : lando
2020-04-09 05:28:34 +00:00
Sylvestre Ledru 205bea23c3 Bug 1617528 - Enable modernize-use-using at review phase r=andi
Differential Revision: https://phabricator.services.mozilla.com/D63819

--HG--
extra : moz-landing-system : lando
2020-02-24 10:05:16 +00:00
Sylvestre Ledru f9328d2021 Bug 1613526 - Create a code quality documentation and move the appropriate docs r=ahal
This for a few reasons:
* The summary becomes the landing page for code quality:
  https://firefox-source-docs.mozilla.org/tools/static-analysis/summary.html
* I don't think we need a full code quality category
* Closer to the source-code-doc
* All the files at the same place

Differential Revision: https://phabricator.services.mozilla.com/D61767

--HG--
rename : tools/lint/docs/coding-style/coding_style_java.rst => docs/code-quality/coding-style/coding_style_java.rst
rename : tools/lint/docs/coding-style/coding_style_other.rst => docs/code-quality/coding-style/coding_style_other.rst
rename : tools/lint/docs/coding-style/coding_style_python.rst => docs/code-quality/coding-style/coding_style_python.rst
rename : tools/lint/docs/coding-style/format_cpp_code_with_clang-format.rst => docs/code-quality/coding-style/format_cpp_code_with_clang-format.rst
rename : tools/clang-tidy/docs/summary.rst => docs/code-quality/index.rst
rename : tools/lint/docs/create.rst => docs/code-quality/lint/create.rst
rename : tools/lint/docs/index.rst => docs/code-quality/lint/index.rst
rename : tools/lint/docs/index.rst => docs/code-quality/lint/lint.rst
rename : tools/lint/docs/linters/codespell.rst => docs/code-quality/lint/linters/codespell.rst
rename : tools/lint/docs/linters/cpp-virtual-final.rst => docs/code-quality/lint/linters/cpp-virtual-final.rst
rename : tools/lint/docs/linters/eslint-plugin-mozilla.rst => docs/code-quality/lint/linters/eslint-plugin-mozilla.rst
rename : tools/lint/docs/linters/eslint-plugin-spidermonkey-js.rst => docs/code-quality/lint/linters/eslint-plugin-spidermonkey-js.rst
rename : tools/lint/docs/linters/eslint.rst => docs/code-quality/lint/linters/eslint.rst
rename : tools/lint/docs/linters/file-perm.rst => docs/code-quality/lint/linters/file-perm.rst
rename : tools/lint/docs/linters/file-whitespace.rst => docs/code-quality/lint/linters/file-whitespace.rst
rename : tools/lint/docs/linters/flake8.rst => docs/code-quality/lint/linters/flake8.rst
rename : tools/lint/docs/linters/l10n.rst => docs/code-quality/lint/linters/l10n.rst
rename : tools/lint/docs/linters/license.rst => docs/code-quality/lint/linters/license.rst
rename : tools/lint/docs/linters/lintpref.rst => docs/code-quality/lint/linters/lintpref.rst
rename : tools/lint/docs/linters/mingw-capitalization.rst => docs/code-quality/lint/linters/mingw-capitalization.rst
rename : tools/lint/docs/linters/perfdocs.rst => docs/code-quality/lint/linters/perfdocs.rst
rename : tools/lint/docs/linters/rstlinter.rst => docs/code-quality/lint/linters/rstlinter.rst
rename : tools/lint/docs/linters/rustfmt.rst => docs/code-quality/lint/linters/rustfmt.rst
rename : tools/lint/docs/usage.rst => docs/code-quality/lint/usage.rst
rename : tools/clang-tidy/docs/index.rst => docs/code-quality/static-analysis.rst
extra : moz-landing-system : lando
2020-02-11 09:11:44 +00:00
Csoregi Natalia 8b571b9067 Backed out changeset f65bd225f199 (bug 1609864) as requested. a=backout DONTBUILD 2020-02-03 10:34:00 +02:00
Andi-Bogdan Postelnicu 678216c088 Bug 1609864 - For `clang-tidy` disable checker `performance-noexcept-move-constructor`. r=sylvestre
Since we don't build use exceptions nor we use STL containers that rely on this when choosing copy
 from move ctors, if these are not marked with noexcept, we should disable this checker since
 it brings only noise to our analysis.

Differential Revision: https://phabricator.services.mozilla.com/D60218

--HG--
extra : moz-landing-system : lando
2020-01-17 08:49:01 +00:00
Andi-Bogdan Postelnicu 43270f5c15 Bug 1606630 - add tests with `if constexpr` for checkers `readability-misleading-indentation` and `readability-braces-around-statements`. r=sylvestre
Differential Revision: https://phabricator.services.mozilla.com/D58936

--HG--
extra : moz-landing-system : lando
2020-01-08 08:39:44 +00:00
Sylvestre Ledru 25a4df6b51 Bug 1606170 - Create a list of static analyzer/linters r=andi
Differential Revision: https://phabricator.services.mozilla.com/D58325

--HG--
extra : moz-landing-system : lando
2020-01-02 16:55:33 +00:00
Sylvestre Ledru 1e63d1f7ca Bug 1605852 - Disable performance-inefficient-algorithm as the test isn't C++17 ready r=andi CLOSED TREE
Differential Revision: https://phabricator.services.mozilla.com//D58529

--HG--
extra : histedit_source : b6e9096717acf79065eb09d661ab11dc8cd8407b
2020-01-02 17:24:07 +02:00
Sylvestre Ledru 3ff277ca7c Bug 1605852 - Enable modernize-concat-nested-namespaces r=andi
Depends on D58190

Differential Revision: https://phabricator.services.mozilla.com/D58191

--HG--
extra : moz-landing-system : lando
2019-12-26 10:22:23 +00:00
Sylvestre Ledru bfaa7ac663 Bug 1606452 - Update the version of clang-format in the configuration r=andi
Differential Revision: https://phabricator.services.mozilla.com/D58456

--HG--
extra : moz-landing-system : lando
2019-12-31 19:07:01 +00:00
Jeff Gilbert 1e61298151 Bug 1601103 - Enable clang-tidy's cppcoreguidelines-pro-type-member-init. r=andi
Differential Revision: https://phabricator.services.mozilla.com/D55745

--HG--
extra : moz-landing-system : lando
2019-12-04 19:57:08 +00:00
Sylvestre Ledru 19d21ac142 no bug - Add links to the static analysis project in the doc r=andi
Differential Revision: https://phabricator.services.mozilla.com/D55465

--HG--
extra : moz-landing-system : lando
2019-12-02 17:04:27 +00:00
Sylvestre Ledru e8c4cdd941 Bug 1600361 - Improve the static analysis doc r=andi
Differential Revision: https://phabricator.services.mozilla.com/D55349

--HG--
extra : moz-landing-system : lando
2019-11-30 11:31:48 +00:00
Sylvestre Ledru 09ed42e0fd Bug 1600361 - Move the static analysis doc in tree r=andi
+ add a thing about coding style

Differential Revision: https://phabricator.services.mozilla.com/D55345

--HG--
extra : moz-landing-system : lando
2019-11-30 10:18:37 +00:00
Andi-Bogdan Postelnicu c85e350b5b Bug 1599101 - For clang-tidy based analysis add checker `clang-analyzer-cplusplus.Move`. r=sylvestre
Differential Revision: https://phabricator.services.mozilla.com/D54522

--HG--
extra : moz-landing-system : lando
2019-11-25 15:54:58 +00:00
Andi-Bogdan Postelnicu a907930749 Bug 1558359 - clang-tidy - For `performance-move-const-arg` disable detection of trivially copyable types that do not have a move constructor. r=sylvestre
Differential Revision: https://phabricator.services.mozilla.com/D49415

--HG--
extra : moz-landing-system : lando
2019-10-16 12:59:33 +00:00
David Major 103a1cbba6 Bug 1573211 - Update to clang 9.0.0 r=glandium
Updates all clang 8.0.1 to version 9.0.0, except for the mingw builds
which suffer from bug 1548624 and will be handled separately later.

Differential Revision: https://phabricator.services.mozilla.com/D44160

MANUAL PUSH: Rebuild clang toolchains without stalling autoland

--HG--
rename : build/build-clang/clang-8-android.json => build/build-clang/clang-android.json
rename : build/build-clang/clang-8-linux64-aarch64-cross.json => build/build-clang/clang-linux64-aarch64-cross.json
rename : build/build-clang/clang-8-linux64.json => build/build-clang/clang-linux64.json
rename : build/build-clang/clang-8-macosx64.json => build/build-clang/clang-macosx64.json
rename : taskcluster/scripts/misc/build-clang-8-linux-macosx-cross.sh => taskcluster/scripts/misc/build-clang-linux-macosx-cross.sh
2019-09-21 16:26:53 +02:00
Brindusan Cristian e2348a18ee Backed out 2 changesets (bug 1573211) as requested by dmajor on irc. CLOSED TREE
Backed out changeset 587463567434 (bug 1573211)
Backed out changeset de0fe40466cb (bug 1573211)

--HG--
rename : build/build-clang/clang-android.json => build/build-clang/clang-8-android.json
rename : build/build-clang/clang-linux64-aarch64-cross.json => build/build-clang/clang-8-linux64-aarch64-cross.json
rename : build/build-clang/clang-linux64.json => build/build-clang/clang-8-linux64.json
rename : build/build-clang/clang-macosx64.json => build/build-clang/clang-8-macosx64.json
rename : taskcluster/scripts/misc/build-clang-linux-macosx-cross.sh => taskcluster/scripts/misc/build-clang-8-linux-macosx-cross.sh
extra : histedit_source : 3f9570ab67fd42186265b1dbb6e93c8342bc60e2
2019-09-26 20:12:51 +03:00
Andi-Bogdan Postelnicu 5f02cc9b02 MANUAL PUSH: Fix for Bug 1573211 - Re-generate the autotest results for clang-tidy analysis. r=myself.
--HG--
extra : rebase_source : d360e92b0da5701e1cfd4e1f49032283bd288ed5
extra : amend_source : edf7899f8bce61f4a5120713bd04cedc83085ea7
2019-09-25 17:30:19 +03:00