summaryrefslogtreecommitdiffstats
path: root/Source
Commit message (Collapse)AuthorAgeFilesLines
* Merge topic 'cmprop-getglobalprop'Brad King2020-03-2719-83/+95
|\ | | | | | | | | | | | | c84cf42897 cmState::GetGlobalProperty: return cmProp Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4521
| * cmState::GetGlobalProperty: return cmPropVitaly Stakhovsky2020-03-2519-83/+95
| |
* | Merge topic 'cmprop-state'Brad King2020-03-2711-29/+32
|\ \ | | | | | | | | | | | | | | | | | | a7f2ff16a4 cmState::GetCacheEntryProperty: return cmProp Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4522
| * | cmState::GetCacheEntryProperty: return cmPropVitaly Stakhovsky2020-03-2511-29/+32
| | |
* | | Merge topic 'export-error-on-multuple-calls'Brad King2020-03-272-1/+27
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 0cd20e8f62 export(): raise an error on multiple calls with same FILE Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4510
| * | | export(): raise an error on multiple calls with same FILEMarc Chevrier2020-03-262-1/+27
| | | | | | | | | | | | | | | | Fixes: 20472
* | | | Merge topic 'ctest-timeout-report'Brad King2020-03-272-8/+14
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3503a79639 Merge branch 'backport-ctest-timeout-report' into ctest-timeout-report 7fda917fa4 CTest: Fix reported duration on timeout when grindchild keeps pipes open Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4530
| * \ \ \ Merge branch 'backport-ctest-timeout-report' into ctest-timeout-reportBrad King2020-03-262-8/+14
| |\ \ \ \
| | * | | | CTest: Fix reported duration on timeout when grindchild keeps pipes openBrad King2020-03-262-8/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since commit d1976cd1f2 (CTest: Fix timeout when grandchild keeps pipes open, 2020-01-13, v3.17.0-rc1~169^2) we no longer hang, but the test duration we report after the timeout is the amount of time the immediate child ran before exiting. Fix the logic to instead report the actual amount of time we spent monitoring the test before the timeout. Fixes: #20509
* | | | | | Merge topic 'llvm-rc-stderr'Brad King2020-03-271-0/+9
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | dc93cbb0d8 llvm-rc: Print stderr output when calling tools through cmake_llvm_rc Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4525
| * | | | | | llvm-rc: Print stderr output when calling tools through cmake_llvm_rcThomas Bernard2020-03-261-0/+9
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | The stored error pipe is output if the program fails. Fixes: #20494
* | | | | | Merge topic 'make-nested-silent'Brad King2020-03-271-1/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | d6d9da5178 Makefiles: Fix silencing of nested calls for GNU make 4.3 Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4515
| * | | | | | Makefiles: Fix silencing of nested calls for GNU make 4.3Brad King2020-03-261-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since GNU make 4.3, `.SILENT:` no longer causes nested `$(MAKE)` calls to get `-s` implicitly. Add the `-s` flag explicitly on such calls to suppress messages about Entering/Leaving directories. Fixes: #20487
* | | | | | | CMake Nightly Date StampKitware Robot2020-03-271-1/+1
| |_|/ / / / |/| | | | |
* | | | | | Merge topic 'cleanup-endls-3'Brad King2020-03-264-124/+121
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8e3a65d963 Refactor: Avoid `std::endl` where it's not necessary (part 3) Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4517
| * | | | | | Refactor: Avoid `std::endl` where it's not necessary (part 3)Alex Turbov2020-03-244-124/+121
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `std::endl` manipulator, except inserting `\n` character, also performs `os.flush()`, which may lead to undesired effects (like disk I/O in the middle of forming data strings). For the `std::stringstream` it also has no meaning. * replace multiple `operator<<` calls on a string literal w/ the only call and the only (bigger) string literal; * replace one character string literal used in `operator<<` w/ a char literal.
* | | | | | | Merge topic 'cleanup-endls-2'Brad King2020-03-268-93/+85
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1e4b5c7d09 Refactor: Avoid `std::endl` where it's not necessary (part 2) Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4514
| * | | | | | | Refactor: Avoid `std::endl` where it's not necessary (part 2)Alex Turbov2020-03-248-93/+85
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `std::endl` manipulator, except inserting `\n` character, also performs `os.flush()`, which may lead to undesired effects (like disk I/O in the middle of forming data strings). For the `std::stringstream` it also has no meaning. * replace multiple `operator<<` calls on a string literal w/ the only call and the only (bigger) string literal; * replace one character string literal used in `operator<<` w/ a char literal.
* | | | | | | Merge topic 'string-prefix'Brad King2020-03-2661-512/+490
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ec7928ef26 use _s to construct static string_views at several places 94de927cab VS10Generator: avoid many string allocations 8ca2504a4d use string_views to avoid memory allocations 761f1adcae check for a valid URL scheme before starting to do any splitting ef778d77e0 replace std::string::substr() with operations that do not allocate memory 77616f4681 pass cm::string_view to cmVisualStudioSlnParser::ParseTag() ada6a3226f use cm::string_view for language extension lookups 48adc29721 replace "std::string::find(x) == 0" with cmHasPrefix() ... Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4501
| * | | | | | | use _s to construct static string_views at several placesRolf Eike Beer2020-03-244-106/+116
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This should avoid the runtime strlen() call.
| * | | | | | | VS10Generator: avoid many string allocationsRolf Eike Beer2020-03-242-48/+36
| | | | | | | |
| * | | | | | | use string_views to avoid memory allocationsRolf Eike Beer2020-03-246-25/+37
| | | | | | | |
| * | | | | | | check for a valid URL scheme before starting to do any splittingRolf Eike Beer2020-03-231-6/+7
| | | | | | | |
| * | | | | | | replace std::string::substr() with operations that do not allocate memoryRolf Eike Beer2020-03-2310-20/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Modify the original string instead of creating a new copy with substr() when it is not used for anything else afterwards.
| * | | | | | | pass cm::string_view to cmVisualStudioSlnParser::ParseTag()Rolf Eike Beer2020-03-232-7/+8
| | | | | | | |
| * | | | | | | use cm::string_view for language extension lookupsRolf Eike Beer2020-03-234-12/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Once the list of extensions is build the set is just a copy of the vector and not modified anymore. Use a string_view for the members of the set, which saves a small amount of memory. It also makes possible to use string_views as lookup keys, so the callers do not need to create copies for the extensions anymore.
| * | | | | | | replace "std::string::find(x) == 0" with cmHasPrefix()Rolf Eike Beer2020-03-2327-114/+129
| | | | | | | |
| * | | | | | | replace "substr(0, xx) ==" with cmHasPrefix()Rolf Eike Beer2020-03-239-24/+16
| | | | | | | |
| * | | | | | | allow cmCTest::CleanString() to skip input charactersRolf Eike Beer2020-03-232-4/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This entirely avoids that one needs to call std::string::substr() for the input.
| * | | | | | | CTest: avoid repeated string comparesRolf Eike Beer2020-03-232-132/+90
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Only one key can match per iteration, avoid any further compares when one match was already found. While at it entirely avoid that the key and value strings are copied.
| * | | | | | | Mumps coverage: directly pass std::string as argumentRolf Eike Beer2020-03-236-9/+9
| | | | | | | |
| * | | | | | | remove pointless return value from cmCTestTestHandler::CleanTestOutput()Rolf Eike Beer2020-03-232-4/+3
| | | | | | | |
| * | | | | | | PyCoverage: avoid repeated string splitting, especially for uncovered linesRolf Eike Beer2020-03-231-20/+17
| | | | | | | |
| * | | | | | | use std::string::rfind() instead of open coding itRolf Eike Beer2020-03-231-12/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | While at it avoid creating a new string.
| * | | | | | | CPackWIXGenerator: use cmStrCat for more partsRolf Eike Beer2020-03-231-5/+5
| | | | | | | |
| * | | | | | | remove needless check for std::string::substr()Rolf Eike Beer2020-03-231-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Passing npos is legal and means "rest of the string".
| * | | | | | | cmcldeps: replace inefficient std::string::substr usageRolf Eike Beer2020-03-231-26/+24
| | | | | | | |
| * | | | | | | replace private startsWith() implementation with cmHasPrefix()Rolf Eike Beer2020-03-231-8/+4
| | | | | | | |
* | | | | | | | Merge topic 'update-kwsys'Brad King2020-03-265-88/+116
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 5f8d1668a2 Merge branch 'upstream-KWSys' into update-kwsys 9d3b9ec4ab KWSys 2020-03-25 (4380f1ae) Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4509
| * | | | | | | | Merge branch 'upstream-KWSys' into update-kwsysBrad King2020-03-255-88/+116
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | # By KWSys Upstream * upstream-KWSys: KWSys 2020-03-25 (4380f1ae)
* | | | | | | | | Merge topic 'compile-commands-collapse-whitespace'Brad King2020-03-263-4/+16
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | e565053bce Ninja: Remove unnecessary newlines in compile commands 5d4bab500e Avoid consecutive whitespace in rules d8622fbd0f Modules: Collapse consecutive whitespace in strings Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4512
| * | | | | | | | | Ninja: Remove unnecessary newlines in compile commandsDaan De Meyer2020-03-231-2/+2
| | | | | | | | | |
| * | | | | | | | | Avoid consecutive whitespace in rulesDaan De Meyer2020-03-232-2/+14
| | |_|/ / / / / / | |/| | | | | | |
* | | | | | | | | Merge topic 'cleanup-CollapseFullPath-calls'Brad King2020-03-2614-104/+52
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 25f48761fa Simplify absolute path conversions using CollapseFullPath full signature 12b39aef75 Remove redundant calls to CollapseFullPath Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4518
| * | | | | | | | Simplify absolute path conversions using CollapseFullPath full signatureBrad King2020-03-247-51/+19
| | | | | | | | |
| * | | | | | | | Remove redundant calls to CollapseFullPathBrad King2020-03-248-53/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove calls where it is known the input is already a collapsed full path.
* | | | | | | | | CMake Nightly Date StampKitware Robot2020-03-261-1/+1
| |_|_|_|_|_|/ / |/| | | | | | |
* | | | | | | | Merge topic 'cmake-compilation-no-compiler-extensions'Brad King2020-03-256-0/+53
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | f034b0f663 CMake compilation: do not use compiler extensions Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4477
| * | | | | | | | CMake compilation: do not use compiler extensionsMarc Chevrier2020-03-236-0/+53
| | |/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For now, compiler extensions are no longer activated on CMake sources. However these extensions are still used for various third parties. This MR is a partial answer to the issue #20454.
* | | | | | | | Merge topic 'cmprop-state'Brad King2020-03-2510-55/+66
|\ \ \ \ \ \ \ \ | |_|_|_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ef408dd232 cmState::GetTargetTypeName: return type is *cmProp Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4505