summaryrefslogtreecommitdiffstats
path: root/Source
Commit message (Collapse)AuthorAgeFilesLines
* Merge topic 'clang-tidy-module-pragma-once-check'Brad King2022-11-183-1/+5
|\ | | | | | | | | | | | | | | | | | | | | | | 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
| |
* | Merge topic 'ninja-swiftmodule-no-exe'Brad King2022-11-181-1/+3
|\ \ | | | | | | | | | | | | | | | | | | 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-171-1/+3
| | | | | | | | | | | | | | | | | | 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 '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-182-16/+39
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-172-1/+13
| | | | | | | | | | | | | | | | | | | | Fixes: #23209
| * | | | cmGeneratorExpression: Refactor stripExportInterface() to use enum classKyle Edwards2022-11-171-16/+27
| | |_|/ | |/| |
* | | | Merge topic 'genex-TARGET_PROPERTY-scope'Brad King2022-11-181-1/+4
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8437141b53 Genex: Fix TARGET_PROPERTY lookup scope in transitive usage requirements Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Acked-by: Robert Maynard <robertjmaynard@gmail.com> Merge-request: !7928
| * | | | Genex: Fix TARGET_PROPERTY lookup scope in transitive usage requirementsBrad King2022-11-171-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When `$<TARGET_PROPERTY:tgt,prop>` is used in an `INTERFACE_*` target property for usage requirements, it may be evaluated in the context of a dependent target in another directory. Look up the `tgt` name in the directory of the target whose property holds the expression so that imported targets isolated to that directory are visible. Fixes: #24163
* | | | | CMake Nightly Date StampKitware Robot2022-11-181-1/+1
| |/ / / |/| | |
* | | | Merge topic 'vs-build-parallel'Brad King2022-11-171-2/+0
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 48db261e69 cmake --build: Use both inter- and intra-project parallelism with msbuild Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !7926
| * | | | cmake --build: Use both inter- and intra-project parallelism with msbuildBrad King2022-11-161-2/+0
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since commit 1ab3881ec9 (cmake: Add options for parallel builds to --build mode, 2018-04-14, v3.12.0-rc1~42^2), `cmake --build --parallel` has added two options to the msbuild command line: * `/m` enables inter-project parallelism in msbuild. * `/p:CL_MPCount=1` suppresses intra-project parallelism in cl. The latter was used to avoid `O(N^2)` compilation threads on `N` processors. In practice however, projects often have many source files in a few targets, so `--parallel` actually reduces parallelism. Drop the latter option to restore intra-project parallelism. Users can always add `-- /p:CL_MPCount=1` themselves. Fixes: #20564
generated by cgit v0.12 at 2025-01-07 22:43:45 (GMT)