Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | cmCTestMultiProcessHandler: Explain testRun ownership in comments | Brad King | 2019-08-26 | 1 | -0/+3 |
| | | | | | | | | The ownership semantics of the 'testRun' variable are subtle and may fool static analysers. Add comments explaining them for now. Later some refactoring could be done to clarify the code. Issue: #19610 | ||||
* | CTest: Simplify some boolean conditions | Brad King | 2019-08-26 | 2 | -4/+2 |
| | | | | | | | Directly compare two boolean values instead of spelling out accepted combinations. Issue: #19610 | ||||
* | cmMessageCommand: Remove extra layer of parentheses | Brad King | 2019-08-26 | 1 | -2/+2 |
| | | | | | | | | | The logic checking `CMAKE_WARN_DEPRECATED` contained an unnecessary layer of parentheses. The condition is of the form `!IsSet || IsOn` which is correct because the documentation says that the behavior is enabled unless the variable is explicitly set to a false value. Issue: #19610 | ||||
* | cmVSSetupHelper: Remove unused SmartBSTR copy operations | Brad King | 2019-08-26 | 1 | -22/+2 |
| | | | | | For our use case we do not actually need to copy these. Mark the operations as `= delete` to simplify the code. | ||||
* | cmVSSetupHelper: Fix SmartBSTR copy operations | Brad King | 2019-08-26 | 1 | -2/+4 |
| | | | | | | | | | Fix the SmartBSTR copy constructor and copy assignment operator added by commit 18c8278b62 (VS: Add helper class to interact with Visual Studio Installer, 2016-12-14, v3.8.0-rc1~93^2~4) to use the string from the source of the copy. Issue: #19610 | ||||
* | cmMakefileTargetGenerator: Check for null before using a pointer | Brad King | 2019-08-26 | 1 | -4/+4 |
| | | | | | | | | Fix the order of logic added by commit 7740ccd1a4 (ENH: some cleanup of the makefile generator, 2006-02-14, v2.4.0~517) to check for allocation failure ('new' returns null) before using the pointer. Issue: #19610 | ||||
* | cmGlobalVisualStudioGenerator: Fix buffer sizes used with RegEnumKeyExW | Brad King | 2019-08-22 | 1 | -4/+4 |
| | | | | | | | | | | | In commit 0b9906c2fb (Windows: Use wide-character system APIs, 2013-12-04, v3.0.0-rc1~254^2) several buffer size computations had to be updated to multiply by `sizeof(wchar_t)`, but for RegEnumKeyExW we were already computing the correct number of characters with a division which was accidentally converted to a multiplication. Use `cm::size` to compute the number of characters in the buffer instead. Issue: #19610 | ||||
* | cmGlobalVisualStudioGenerator: Fix buffer sizes used RegQueryValueExW | Brad King | 2019-08-22 | 1 | -5/+4 |
| | | | | | | | | | In commit 0b9906c2fb (Windows: Use wide-character system APIs, 2013-12-04, v3.0.0-rc1~254^2) several buffer size computations had to be updated to multiply by `sizeof(wchar_t)`, but some for RegQueryValueExW were incorrect because the number of bytes was already computed. Issue: #19610 | ||||
* | Merge topic 'cmCommand_refactor' | Brad King | 2019-08-22 | 70 | -2042/+1568 |
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6ab28b9413 cmCommand refactor: cmStringCommand 36f32d3604 cmCommand refactor: cmSetPropertyCommand 7c83c19205 cmCommand refactor: cmSetDirectoryPropertiesCommand 9413952c42 cmCommand refactor: cmCMakePolicyCommand 07ea93de54 cmCommand refactor: cmWriteFileCommand ca3b9186bb cmCommand refactor: cmVariableWatchCommand b1acc711f4 cmCommand refactor: cmRemoveCommand 413a960391 cmCommand refactor: cmCMakeHostSystemInformationCommand ... Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !3673 | ||||
| * | cmCommand refactor: cmStringCommand | Gabor Bencze | 2019-08-20 | 3 | -221/+241 |
| | | |||||
| * | cmCommand refactor: cmSetPropertyCommand | Gabor Bencze | 2019-08-20 | 3 | -198/+256 |
| | | |||||
| * | cmCommand refactor: cmSetDirectoryPropertiesCommand | Gabor Bencze | 2019-08-20 | 3 | -40/+21 |
| | | |||||
| * | cmCommand refactor: cmCMakePolicyCommand | Gabor Bencze | 2019-08-20 | 4 | -77/+66 |
| | | |||||
| * | cmCommand refactor: cmWriteFileCommand | Gabor Bencze | 2019-08-20 | 3 | -33/+11 |
| | | |||||
| * | cmCommand refactor: cmVariableWatchCommand | Gabor Bencze | 2019-08-20 | 3 | -31/+12 |
| | | |||||
| * | cmCommand refactor: cmRemoveCommand | Gabor Bencze | 2019-08-20 | 3 | -30/+9 |
| | | |||||
| * | cmCommand refactor: cmCMakeHostSystemInformationCommand | Gabor Bencze | 2019-08-20 | 3 | -82/+57 |
| | | |||||
| * | cmCommand refactor: cmUnsetCommand | Gabor Bencze | 2019-08-20 | 3 | -33/+12 |
| | | |||||
| * | cmCommand refactor: cmSiteNameCommand | Gabor Bencze | 2019-08-20 | 3 | -32/+11 |
| | | |||||
| * | cmCommand refactor: cmSetCommand | Gabor Bencze | 2019-08-20 | 3 | -39/+18 |
| | | |||||
| * | cmCommand refactor: cmSeparateArgumentsCommand | Gabor Bencze | 2019-08-20 | 3 | -34/+12 |
| | | |||||
| * | cmCommand refactor: cmParseArgumentsCommand | Gabor Bencze | 2019-08-20 | 3 | -47/+24 |
| | | |||||
| * | cmCommand refactor: cmOptionCommand | Gabor Bencze | 2019-08-20 | 3 | -38/+17 |
| | | |||||
| * | cmCommand refactor: CmMessageCommand | Gabor Bencze | 2019-08-20 | 3 | -40/+20 |
| | | |||||
| * | cmCommand refactor: cmMathCommand | Gabor Bencze | 2019-08-20 | 3 | -41/+25 |
| | | |||||
| * | cmCommand refactor: cmMarkAsAdvancedCommand | Gabor Bencze | 2019-08-20 | 3 | -32/+10 |
| | | |||||
| * | cmCommand refactor: cmMakeDirectoryCommand | Gabor Bencze | 2019-08-20 | 3 | -32/+10 |
| | | |||||
| * | cmCommand refactor: cmListCommand | Gabor Bencze | 2019-08-20 | 3 | -224/+244 |
| | | |||||
| * | cmCommand refactor: cmMacroCommand | Gabor Bencze | 2019-08-20 | 3 | -27/+10 |
| | | |||||
| * | cmCommand refactor: cmIncludeGuardCommand | Gabor Bencze | 2019-08-20 | 3 | -31/+10 |
| | | |||||
| * | cmCommand refactor: cmIncludeCommand | Gabor Bencze | 2019-08-20 | 3 | -48/+29 |
| | | |||||
| * | cmCommand_refactor: cmGetPropertyCommand | Gabor Bencze | 2019-08-20 | 3 | -162/+187 |
| | | |||||
| * | cmCommand refactor: cmGetFilenameComponentCommand | Gabor Bencze | 2019-08-20 | 3 | -37/+16 |
| | | |||||
| * | cmCommand refactor: cmGetDirectoryPropertyCommand | Gabor Bencze | 2019-08-20 | 3 | -42/+29 |
| | | |||||
| * | cmCommand refactor: cmGetCMakePropertyCommand | Gabor Bencze | 2019-08-20 | 3 | -31/+12 |
| | | |||||
| * | cmCommand refactor: cmFunctionCommand | Gabor Bencze | 2019-08-20 | 3 | -27/+10 |
| | | |||||
| * | cmCommand refactor: cmForEachCommand | Gabor Bencze | 2019-08-20 | 3 | -39/+21 |
| | | |||||
| * | cmCommand refactor: cmExecuteProcessCommand | Gabor Bencze | 2019-08-20 | 3 | -61/+46 |
| | | |||||
| * | cmCommand refactor: cmExecProgramCommand | Gabor Bencze | 2019-08-20 | 3 | -47/+25 |
| | | |||||
| * | cmCommand refactor: cmContinueCommand | Gabor Bencze | 2019-08-20 | 3 | -33/+15 |
| | | |||||
| * | cmCommand refactor: cmConfigureFileCommand | Gabor Bencze | 2019-08-20 | 3 | -73/+37 |
| | | |||||
| * | cmCommand refactor: cmCMakeMinimumRequired | Gabor Bencze | 2019-08-20 | 3 | -49/+34 |
| | | |||||
| * | cmCommand refactor: cmBreakCommand | Gabor Bencze | 2019-08-20 | 3 | -31/+11 |
| | | |||||
* | | Merge topic 'isolate-declarations' | Brad King | 2019-08-22 | 25 | -48/+102 |
|\ \ | | | | | | | | | | | | | | | | | | | d331021255 clang-tidy: isolate declarations for readability Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !3704 | ||||
| * | | clang-tidy: isolate declarations for readability | Regina Pfeifer | 2019-08-20 | 25 | -48/+102 |
| |/ | |||||
* | | CMake Nightly Date Stamp | Kitware Robot | 2019-08-22 | 1 | -1/+1 |
| | | |||||
* | | Merge topic 'double-export-error-message-install' | Brad King | 2019-08-21 | 4 | -18/+33 |
|\ \ | | | | | | | | | | | | | | | | | | | | | | d17580909f cmExportInstallFileGenerator: improve error message a18100898a cmInstallExportGenerator: add a method for the file destination Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !3703 | ||||
| * | | cmExportInstallFileGenerator: improve error message | Ben Boeckel | 2019-08-20 | 2 | -18/+27 |
| | | | | | | | | | | | | | | | | | | | | | Following commit 49cfd390 (cmExportBuildFileGenerator: improve error message, 2019-06-26), improve the error message related to installed export sets by referencing the files which contain the exported target ambiguously. | ||||
| * | | cmInstallExportGenerator: add a method for the file destination | Ben Boeckel | 2019-08-19 | 2 | -0/+6 |
| | | | | | | | | | | | | | | | This will be used in the error message which remarks about a target being exported ambiguously into different installations. | ||||
* | | | Merge topic 'shared-string' | Brad King | 2019-08-21 | 5 | -28/+33 |
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | be7807478c cmDefinitions: Reduce allocation of keys and values in MakeClosure e07e2bc8bb bootstrap: Compile cm::String c1787cb5eb cpack.cxx: Re-order include blocks to follow our conventions 141e307484 cmConfigure.h: Tell windows.h not to define min/max macros Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !3669 |