summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* clang-tidy module: look for sizeof string literal in cmStrLen() checkKyle Edwards2022-11-184-4/+90
|
* Merge topic 'clang-tidy-module-pragma-once-check'Brad King2022-11-1818-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 checkKyle Edwards2022-11-173-1/+5
| |
| * clang-tidy module: add test for #pragma once checkSean Orner2022-11-1710-0/+79
| |
| * clang-tidy module: add #pragma once checkIgor-Mikhail-Valentin Glebov2022-11-174-0/+389
| | | | | | | | Co-Authored-by: Kyle Edwards <kyle.edwards@kitware.com>
| * clang-tidy module: allow header files in test casesKyle Edwards2022-11-171-12/+42
| |
* | Merge topic 'ninja-swiftmodule-no-exe'Brad King2022-11-185-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 buildEvan Wilde2022-11-175-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 King2022-11-182-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 nvfortranRobert Maynard2022-11-162-1/+4
| | | |
* | | | Merge topic 'refactor-cmDocumentation'Brad King2022-11-1842-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 entryAlex Turbov2022-11-1728-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 valuesAlex Turbov2022-11-171-7/+5
| | | | |
| * | | | cmake.cxx: Simplify `cmake::AppendExtraGeneratorsDocumentation()`Alex Turbov2022-11-171-12/+5
| | | | |
| * | | | cpack.cxx: Optimize calls to `std::osteam::operator<<`Alex Turbov2022-11-171-64/+53
| | | | |
| * | | | cpack.cxx: Eliminate redundant `if`Alex Turbov2022-11-171-5/+1
| | | | |
| * | | | cpack.cxx: Deduplicate "Generators" section creation codeAlex Turbov2022-11-171-21/+24
| | | | |
| * | | | cmake.cxx: Move `static` functions to an anonymous namespaceAlex Turbov2022-11-171-100/+97
| | | | |
| * | | | cmDocumentationEntry: Drop all user provided ctors for C++ >= 14Alex Turbov2022-11-178-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 Turbov2022-11-171-28/+27
| | | | |
| * | | | cmDocumentationFormatter: Hide internal methods into `private` sectionAlex Turbov2022-11-171-4/+5
| | | | |
| * | | | cmDocumentation: Drop unused `SetSections` methodAlex Turbov2022-11-172-8/+0
| | | | |
| * | | | cmDocumentation: `char*[][2]` → `cmDocumentationEntry[N]`Alex Turbov2022-11-1713-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 Turbov2022-11-171-5/+9
| | | | |
| * | | | ccmake.cxx: Use anonymous namespace instead of `static`Alex Turbov2022-11-171-10/+10
| | | | |
| * | | | ctest.cxx: Eliminate redundand trenary operatorAlex Turbov2022-11-173-3/+3
| | | | |
| * | | | ctest.cxx: Optimize `std::ostream::operator<<` callsAlex Turbov2022-11-171-6/+4
| | | | |
| * | | | ctest.cxx: Use anonymous namespace instead of `static` arraysAlex Turbov2022-11-171-5/+6
| | | | |
| * | | | cmakemain.cxx: Remove always true preprocessor `#if` blockAlex Turbov2022-11-171-2/+0
| | | | |
| * | | | cmDocumentation: Accept `Iterable` instead of `vector` on add entriesAlex Turbov2022-11-172-26/+18
| | | | |
| * | | | cmDocumentationSection: Accept `Iterable` instead of `vector` on addAlex Turbov2022-11-171-7/+11
| | | | |
| * | | | cmDocumentationFormatter: Improve `PrintFormatted`Alex Turbov2022-11-171-34/+54
| | | | | | | | | | | | | | | | | | | | Instead of one char at time copy (append), use lines.
| * | | | cmDocumentationFormatter: All printing methods accept stringsAlex Turbov2022-11-173-13/+14
| | | | |
| * | | | cmDocumentationFormatter: Turn `while ()` into `for ()` loopAlex Turbov2022-11-171-15/+23
| | | | |
| * | | | cmDocumentationFormatter: Eliminate one `if` in the `PrintColumn` loopAlex Turbov2022-11-171-7/+2
| | | | |
| * | | | cmDocumentationFormatter: Turn `TextWidth` member into `size_t`Alex Turbov2022-11-172-8/+10
| | | | | | | | | | | | | | | | | | | | Cuz normally required text width can't be negative. It was `int`…
| * | | | cmDocumentationFormatter: Prevent indentation reset side effectAlex Turbov2022-11-171-2/+7
| | | | | | | | | | | | | | | | | | | | | | | | | Fix `cmDocumentationFormatter::PrintColumn()` method to eliminate an indentation reset side effect.
| * | | | cmDocumentation: Drop useless call to formatter's `SetIndent()`Alex Turbov2022-11-171-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 Turbov2022-11-174-27/+33
| | | | | | | | | | | | | | | | | | | | … instead of `std::string`.
| * | | | cmDocumentationFormatter: Use `std::ostream` padding capabitilyAlex Turbov2022-11-171-7/+6
| | | | | | | | | | | | | | | | | | | | … instead of manually print spaces to align fields.
| * | | | cmDocumentationFormatter: `TextIndent` member is `std::string` nowAlex Turbov2022-11-172-20/+17
| | | | | | | | | | | | | | | | | | | | Was `const char*`.
| * | | | cmDocumentationFormatter: `PrintPreformatted` accept stringAlex Turbov2022-11-172-14/+16
| | | | | | | | | | | | | | | | | | | | Also, make it `const` method cuz it's not modify the state.
| * | | | cmDocumentationFormatter.h: No need `virtual` methodsAlex Turbov2022-11-172-11/+3
| | | | | | | | | | | | | | | | | | | | | | | | | Can be added later it really need. Also, drop ctor/dtor to make the class trivially destructable.
| * | | | cmDocumentationEntry.h: Drop unused constructorAlex Turbov2022-11-171-9/+0
| | | | |
| * | | | cmDocumentation: Optimize `std::ostream::operator<<()` callsAlex Turbov2022-11-172-19/+19
| | | | |
| * | | | cmDocumentation.cxx: Use lambda function instead of macroAlex Turbov2022-11-172-46/+41
| | | | |
| * | | | cmDocumentation.cxx: Use anonymous namespace instead of `static`Alex Turbov2022-11-161-3/+5
| | | | |
| * | | | cmDocumentation: Remove MSVC 6 workaroundAlex Turbov2022-11-162-36/+26
| |/ / /
* | | | Merge topic 'build-local-interface-genex'Brad King2022-11-1810-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:...> genexKyle Edwards2022-11-1710-1/+97
| | | | | | | | | | | | | | | | | | | | Fixes: #23209