Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | clang-tidy module: look for sizeof string literal in cmStrLen() check | Kyle Edwards | 2022-11-18 | 4 | -4/+90 |
| | |||||
* | Merge topic 'clang-tidy-module-pragma-once-check' | Brad King | 2022-11-18 | 18 | -13/+515 |
|\ | | | | | | | | | | | | | | | | | | | | | | | 91a7c0b51c CMake: fix violations of #pragma once check aa0c99c55c clang-tidy module: add test for #pragma once check c9af6f2ff6 clang-tidy module: add #pragma once check 68a4d97da7 clang-tidy module: allow header files in test cases Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Acked-by: Ben Boeckel <ben.boeckel@kitware.com> Merge-request: !7933 | ||||
| * | CMake: fix violations of #pragma once check | Kyle Edwards | 2022-11-17 | 3 | -1/+5 |
| | | |||||
| * | clang-tidy module: add test for #pragma once check | Sean Orner | 2022-11-17 | 10 | -0/+79 |
| | | |||||
| * | clang-tidy module: add #pragma once check | Igor-Mikhail-Valentin Glebov | 2022-11-17 | 4 | -0/+389 |
| | | | | | | | | Co-Authored-by: Kyle Edwards <kyle.edwards@kitware.com> | ||||
| * | clang-tidy module: allow header files in test cases | Kyle Edwards | 2022-11-17 | 1 | -12/+42 |
| | | |||||
* | | Merge topic 'ninja-swiftmodule-no-exe' | Brad King | 2022-11-18 | 5 | -1/+16 |
|\ \ | | | | | | | | | | | | | | | | | | | 38c8807c5a Ninja: Avoid re-linking a Swift executable on every build Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !7901 | ||||
| * | | Ninja: Avoid re-linking a Swift executable on every build | Evan Wilde | 2022-11-17 | 5 | -1/+16 |
| | | | | | | | | | | | | | | | | | | Swift doesn't emit swiftmodules for executables, so we shouldn't put it in dependency graph. Ninja sees the "missing" dependency and always tries to rebuild/re-link the target. | ||||
* | | | Merge topic 'nvhpc_fortran_extensions' | Brad King | 2022-11-18 | 2 | -1/+4 |
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | a98a82f24b Fortran: Support cuf file extension when using nvfortran Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !7925 | ||||
| * | | | Fortran: Support cuf file extension when using nvfortran | Robert Maynard | 2022-11-16 | 2 | -1/+4 |
| | | | | |||||
* | | | | Merge topic 'refactor-cmDocumentation' | Brad King | 2022-11-18 | 42 | -802/+663 |
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6e3e8827fa Refactor: `cmGlobalGeneratorFactory::GetDocumentation` returns entry e99a4acbcd cmGlobalGeneratorFactory.h: Simplify returning default values 7daadd304f cmake.cxx: Simplify `cmake::AppendExtraGeneratorsDocumentation()` df2047c2ac cpack.cxx: Optimize calls to `std::osteam::operator<<` f6180485bb cpack.cxx: Eliminate redundant `if` d7c183f35c cpack.cxx: Deduplicate "Generators" section creation code 9c06f0dd71 cmake.cxx: Move `static` functions to an anonymous namespace 69918b07e1 cmDocumentationEntry: Drop all user provided ctors for C++ >= 14 ... Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !7605 | ||||
| * | | | | Refactor: `cmGlobalGeneratorFactory::GetDocumentation` returns entry | Alex Turbov | 2022-11-17 | 28 | -124/+91 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before, a documentation entry was in/out parameter. Now it's a normal return value. This also makes possible to eliminate defaulted default ctor for `cmDocumentationEntry` for C++ 11. Also, simplify `cmake::AppendGlobalGeneratorsDocumentation()`. | ||||
| * | | | | cmGlobalGeneratorFactory.h: Simplify returning default values | Alex Turbov | 2022-11-17 | 1 | -7/+5 |
| | | | | | |||||
| * | | | | cmake.cxx: Simplify `cmake::AppendExtraGeneratorsDocumentation()` | Alex Turbov | 2022-11-17 | 1 | -12/+5 |
| | | | | | |||||
| * | | | | cpack.cxx: Optimize calls to `std::osteam::operator<<` | Alex Turbov | 2022-11-17 | 1 | -64/+53 |
| | | | | | |||||
| * | | | | cpack.cxx: Eliminate redundant `if` | Alex Turbov | 2022-11-17 | 1 | -5/+1 |
| | | | | | |||||
| * | | | | cpack.cxx: Deduplicate "Generators" section creation code | Alex Turbov | 2022-11-17 | 1 | -21/+24 |
| | | | | | |||||
| * | | | | cmake.cxx: Move `static` functions to an anonymous namespace | Alex Turbov | 2022-11-17 | 1 | -100/+97 |
| | | | | | |||||
| * | | | | cmDocumentationEntry: Drop all user provided ctors for C++ >= 14 | Alex Turbov | 2022-11-17 | 8 | -34/+33 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is no need for them cuz: - the last field has a default value - all static instances use 2 arguments convertible to `std::string` - "dynamic" instances used for _Generator_ doc entries access fields diectly using default constructed instance Moreover, compiler may generate move ctor/assign when needed. | ||||
| * | | | | cmake.cxx: Optimize calls to `std::osteam::operator<<` | Alex Turbov | 2022-11-17 | 1 | -28/+27 |
| | | | | | |||||
| * | | | | cmDocumentationFormatter: Hide internal methods into `private` section | Alex Turbov | 2022-11-17 | 1 | -4/+5 |
| | | | | | |||||
| * | | | | cmDocumentation: Drop unused `SetSections` method | Alex Turbov | 2022-11-17 | 2 | -8/+0 |
| | | | | | |||||
| * | | | | cmDocumentation: `char*[][2]` → `cmDocumentationEntry[N]` | Alex Turbov | 2022-11-17 | 13 | -161/+87 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use fixed size arrays of `cmDocumentationEntry` items instead of open arrays of two `char` pointers when describe program options help screens. Also, drop `const char*[][2]` overloads of methods of `cmDocumentation` and `cmDocumentationSection` classes in the sake of generic (template) appenders introduced earlier. | ||||
| * | | | | CMakeSetup.cxx: Use anonymous namespace instead of `static` | Alex Turbov | 2022-11-17 | 1 | -5/+9 |
| | | | | | |||||
| * | | | | ccmake.cxx: Use anonymous namespace instead of `static` | Alex Turbov | 2022-11-17 | 1 | -10/+10 |
| | | | | | |||||
| * | | | | ctest.cxx: Eliminate redundand trenary operator | Alex Turbov | 2022-11-17 | 3 | -3/+3 |
| | | | | | |||||
| * | | | | ctest.cxx: Optimize `std::ostream::operator<<` calls | Alex Turbov | 2022-11-17 | 1 | -6/+4 |
| | | | | | |||||
| * | | | | ctest.cxx: Use anonymous namespace instead of `static` arrays | Alex Turbov | 2022-11-17 | 1 | -5/+6 |
| | | | | | |||||
| * | | | | cmakemain.cxx: Remove always true preprocessor `#if` block | Alex Turbov | 2022-11-17 | 1 | -2/+0 |
| | | | | | |||||
| * | | | | cmDocumentation: Accept `Iterable` instead of `vector` on add entries | Alex Turbov | 2022-11-17 | 2 | -26/+18 |
| | | | | | |||||
| * | | | | cmDocumentationSection: Accept `Iterable` instead of `vector` on add | Alex Turbov | 2022-11-17 | 1 | -7/+11 |
| | | | | | |||||
| * | | | | cmDocumentationFormatter: Improve `PrintFormatted` | Alex Turbov | 2022-11-17 | 1 | -34/+54 |
| | | | | | | | | | | | | | | | | | | | | Instead of one char at time copy (append), use lines. | ||||
| * | | | | cmDocumentationFormatter: All printing methods accept strings | Alex Turbov | 2022-11-17 | 3 | -13/+14 |
| | | | | | |||||
| * | | | | cmDocumentationFormatter: Turn `while ()` into `for ()` loop | Alex Turbov | 2022-11-17 | 1 | -15/+23 |
| | | | | | |||||
| * | | | | cmDocumentationFormatter: Eliminate one `if` in the `PrintColumn` loop | Alex Turbov | 2022-11-17 | 1 | -7/+2 |
| | | | | | |||||
| * | | | | cmDocumentationFormatter: Turn `TextWidth` member into `size_t` | Alex Turbov | 2022-11-17 | 2 | -8/+10 |
| | | | | | | | | | | | | | | | | | | | | Cuz normally required text width can't be negative. It was `int`… | ||||
| * | | | | cmDocumentationFormatter: Prevent indentation reset side effect | Alex Turbov | 2022-11-17 | 1 | -2/+7 |
| | | | | | | | | | | | | | | | | | | | | | | | | | Fix `cmDocumentationFormatter::PrintColumn()` method to eliminate an indentation reset side effect. | ||||
| * | | | | cmDocumentation: Drop useless call to formatter's `SetIndent()` | Alex Turbov | 2022-11-17 | 1 | -1/+0 |
| | | | | | | | | | | | | | | | | | | | | | | | | | The `cmDocumentationFormatter::PrintSection()` method ignores the currently set indentation level and use it's own. | ||||
| * | | | | cmDocumentationFormatter: Turn `TextIndent` member into `size_t` | Alex Turbov | 2022-11-17 | 4 | -27/+33 |
| | | | | | | | | | | | | | | | | | | | | … instead of `std::string`. | ||||
| * | | | | cmDocumentationFormatter: Use `std::ostream` padding capabitily | Alex Turbov | 2022-11-17 | 1 | -7/+6 |
| | | | | | | | | | | | | | | | | | | | | … instead of manually print spaces to align fields. | ||||
| * | | | | cmDocumentationFormatter: `TextIndent` member is `std::string` now | Alex Turbov | 2022-11-17 | 2 | -20/+17 |
| | | | | | | | | | | | | | | | | | | | | Was `const char*`. | ||||
| * | | | | cmDocumentationFormatter: `PrintPreformatted` accept string | Alex Turbov | 2022-11-17 | 2 | -14/+16 |
| | | | | | | | | | | | | | | | | | | | | Also, make it `const` method cuz it's not modify the state. | ||||
| * | | | | cmDocumentationFormatter.h: No need `virtual` methods | Alex Turbov | 2022-11-17 | 2 | -11/+3 |
| | | | | | | | | | | | | | | | | | | | | | | | | | Can be added later it really need. Also, drop ctor/dtor to make the class trivially destructable. | ||||
| * | | | | cmDocumentationEntry.h: Drop unused constructor | Alex Turbov | 2022-11-17 | 1 | -9/+0 |
| | | | | | |||||
| * | | | | cmDocumentation: Optimize `std::ostream::operator<<()` calls | Alex Turbov | 2022-11-17 | 2 | -19/+19 |
| | | | | | |||||
| * | | | | cmDocumentation.cxx: Use lambda function instead of macro | Alex Turbov | 2022-11-17 | 2 | -46/+41 |
| | | | | | |||||
| * | | | | cmDocumentation.cxx: Use anonymous namespace instead of `static` | Alex Turbov | 2022-11-16 | 1 | -3/+5 |
| | | | | | |||||
| * | | | | cmDocumentation: Remove MSVC 6 workaround | Alex Turbov | 2022-11-16 | 2 | -36/+26 |
| |/ / / | |||||
* | | | | Merge topic 'build-local-interface-genex' | Brad King | 2022-11-18 | 10 | -16/+123 |
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 38cbf5e15b Genex: Add $<BUILD_LOCAL_INTERFACE:...> genex 37b5c78688 cmGeneratorExpression: Refactor stripExportInterface() to use enum class Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !7919 | ||||
| * | | | | Genex: Add $<BUILD_LOCAL_INTERFACE:...> genex | Kyle Edwards | 2022-11-17 | 10 | -1/+97 |
| | | | | | | | | | | | | | | | | | | | | Fixes: #23209 |