git.blender.org/blender.git - Unnamed repository; edit this file 'description' to name the repository.

Age	Commit message (Collapse)	Author
2020-10-21	Cleanup: Clang-tidy readability-named-parameter	Ankit Meel
	No functional change.
2020-10-21	Cleanup: Clang-tidy readability-redundant-string-init	Ankit Meel
	No functional change.
2020-10-21	Clang-tidy: disable new warnings.	Ankit Meel
	Until it is decided whether to work on, or ignore these warning, disable them. See T78535 Reviewed By: sergey Differential Revision: https://developer.blender.org/D9281
2020-09-04	Cleanup: Clang-Tidy readability-inconsistent-declaration-parameter-name fix	Sebastian Parborg
	No functional changes
2020-09-04	Cleanup: Clang-Tidy bugprone-copy-constructor-init fix	Sybren A. Stüvel
	No expected functional changes.
2020-09-04	Cleanup: Clang-Tidy, describe current status of rules	Sybren A. Stüvel
	No functional changes.
2020-09-04	Cleanup: Clang-Tidy bugprone-incorrect-roundings fixes	Sybren A. Stüvel
	Should cause no noticeable difference.
2020-09-04	Cleanup: Enable Clang-Tidy readability-qualified-auto rule	Sybren A. Stüvel
	No functional changes. No changes except enabling the rule, even.
2020-09-04	Cleanup: Clang-Tidy readability-redundant-declaration fix	Sybren A. Stüvel
	No functional changes.
2020-09-04	Cleanup: Clang-Tidy readability-static-accessed-through-instance fixes	Sybren A. Stüvel
	No functional changes.
2020-09-04	Cleanup: Clang-Tidy readability-const-return-type fixes	Sybren A. Stüvel
	No functional changes.
2020-09-04	Cleanup: Clang-Tidy readability-redundant-preprocessor fixes	Sybren A. Stüvel
	Remove redundantly nested `#if` and `#ifdef` statements. One nested `#if 0` block was left untouched, as it's in particle code that's no longer maintained. Furthermore, that block also has some explanation as to the differences between the enabled & disabled parts. One nested `#if 0` construct was completely removed, leaving only the actually used bit of code. There was no explanation as to the usefulness of the disabled code, and it hasn't been touched in years. No functional changes.
2020-08-07	Cleanup: enable Clang-Tidy `bugprone-parent-virtual-call` rule	Sybren A. Stüvel
	I added a single `NOLINT` exception with explanation. No functional changes.
2020-08-07	Cleanup: enabled Clang-Tidy `bugprone-suspicious-missing-comma` rule	Sybren A. Stüvel
	No functional changes because no code changed.
2020-08-07	Cleanup: enabled Clang-Tidy `bugprone-not-null-terminated-result` rule	Sybren A. Stüvel
	No functional changes because no code changed.
2020-08-07	Cleanup: fixed Clang-Tidy `bugprone-suspicious-string-compare` warnings	Sybren A. Stüvel
	No functional changes.
2020-08-07	Cleanup: enabled Clang-Tidy rule `bugprone-infinite-loop`	Sybren A. Stüvel
	Enabling it and doing a full rebuild didn't cause any warnings, so nothing else to do here. No functional changes.
2020-08-07	Cleanup: enable and silence Clang-Tidy readability-function-size	Sybren A. Stüvel
	Enable Clang-Tidy's `readability-function-size` rule and add a few `NOLINT` markers to explicitly silence warnings for three functions. These functions are huge and would IMO benefit from splitting up, but are hard to without intimate knowledge of the code. At least by enabling the rule, we can start tweaking the values and refactoring other functions that bubble up as being too long/complex. No functional changes.
2020-08-07	Cleanup: Clang-Tidy else-after-return fixes	Sybren A. Stüvel
	This addresses warnings from Clang-Tidy's `readability-else-after-return` rule. This should be the final commit of the series of commits that addresses this particular rule. No functional changes.
2020-08-07	Clang-Tidy: Address readability-redundant-string-init	Sergey Sharybin

2020-07-15	Clang-Tidy: ignore some newer checks	Jacques Lucke
	Those checks have been added to clang tidy within the last year. They fail when I use a clang tidy version I built from source. Reviewers: sergey Differential Revision: https://developer.blender.org/D8302
2020-07-13	Clang Tidy: enable readability-non-const-parameter warning	Jacques Lucke
	Clang Tidy reported a couple of false positives. I disabled those `NOLINTNEXTLINE`. Differential Revision: https://developer.blender.org/D8199
2020-07-03	Clang-tidy: Enable braces-around-statements warning	Sebastian Parborg

2020-07-03	Clang-Tidy: Enable bugprone-misplaced-widening-cast	Sergey Sharybin

2020-07-03	Clang-Tidy: enable readability-named-parameter	Jacques Lucke

2020-07-03	Clang-Tidy: enable bugprone-too-small-loop-variable	Jacques Lucke

2020-07-03	Clang-Tidy: enable bugprone-assert-side-effect	Jacques Lucke
	Looks like we have no assertions with side effects.
2020-07-03	Clang-Tidy: enable bugprone-argument-comment	Jacques Lucke
	It was called `inverted` in the header.
2020-07-03	Clang-Tidy: Fix readability-delete-null-pointer warnings	Sergey Sharybin
	Also enable it in the configuration.
2020-07-03	Clang-Tidy: Enable readability-redundant-string-cstr	Jacques Lucke

2020-07-03	Clang-Tidy: Enable readability-redundant-control-flow	Hans Goudey

2020-07-03	Clang-tidy: enable readability-container-size-empty warning	Jacques Lucke
	Reviewers: sergey Differential Revision: https://developer.blender.org/D8197
2020-07-03	Clang-Tidy: Enable redundant static qualifier warning	Sergey Sharybin
	This change enables readability-static-definition-in-anonymous-namespace warning in .clang-tidy configuration.
2020-07-03	Clang-tidy: Enable bugprone-lambda-function-name warning	Sergey Sharybin

2020-07-03	Clang-Tidy: Silence warning affecting newer code	Sergey Sharybin
	For now Clang-Tidy should be passable with the codebase as is. This warning will be addressed as quality day task.
2020-07-03	Initial support of clang-tidy toolchain	Sergey Sharybin
	Clang Tidy is a Clang based "linter" tool which goal is to help fixing typical programming errors. It is run as a separate compile step of every file, which slows compilation down but allows to fully analyze the file the same way as compiler does and catch non-trivial bugprone cases. This change includes: - CMake option called `WITH_CLANG_TIDY` which enables Clang Tidy linter tool on all source in the `source/` directory. This option is only available on Linux, as it is currently the easiest platform to get the Clang Tidy toolchain to work. - CMake module which is aimed to find latest available Clang Tidy. - Set of rules which allows to have Blender fully compiled without extra issues. The goal of this change is to provide a base ground so that solving all the warnings can happen later on, as a team effort. It should be possible to use Clang Tidy side-by-side with both GCC and Clang, but there seems to be some tweaks to be done in CMake to make it really work for Blender. For now use Clang toolchain if there are issues with GCC+Clang Tidy. It will be worked on in the nearest future to bring seamless experience for all configurations. Currently there is no official way of getting Clang Tidy on macOS, and on Windows there are some difficulties of hooking up Clang Tidy from LLVM package to the MSVC compiler toolchain. The actual warnings in the code will be addressed as a part of the Code Quality Days, task T78535. Differential Revision: https://developer.blender.org/D7937