summaryrefslogtreecommitdiffstats
path: root/Source
Commit message (Collapse)AuthorAgeFilesLines
* Merge topic 'cmOutputConverter-stringstream'Brad King2016-06-162-192/+39
|\ | | | | | | | | d645b03e cmOutputConverter: implement Shell__GetArgument using ostringstream
| * cmOutputConverter: implement Shell__GetArgument using ostringstreamDaniel Pfeifer2016-06-152-192/+39
| | | | | | | | | | This removes the need to calculate the resulting string length beforehand.
* | Merge topic 'cleanup-streams'Brad King2016-06-1627-69/+69
|\ \ | | | | | | | | | | | | | | | | | | | | | ed5fa48d cmXMLWriter: use ifstream from KWSys 24ab29b8 Prefer istringstream and ostringstream over stringstream. ab8b77dd Remove redundant arguments from fstream constructors eb79fa72 Access std::ios_base with std::ios
| * | cmXMLWriter: use ifstream from KWSysDaniel Pfeifer2016-06-141-2/+2
| | |
| * | Prefer istringstream and ostringstream over stringstream.Daniel Pfeifer2016-06-1417-55/+55
| | | | | | | | | | | | Use istringsream for parsing, ostringstream for generation.
| * | Remove redundant arguments from fstream constructorsDaniel Pfeifer2016-06-146-8/+8
| | | | | | | | | | | | Don't pass the default value of the openmode parameter explicitly.
| * | Access std::ios_base with std::iosDaniel Pfeifer2016-06-143-4/+4
| | | | | | | | | | | | Just because it is shorter.
* | | Merge topic 'iwyu-preparation'Brad King2016-06-166-84/+163
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1c2c6297 cmTargetLinkLibraryType: Extract from cmStandardIncludes.h ffdc0a8e cmTypeMacro: Extract from cmStandardIncludes.h e4eb88e2 cmCustomCommandLines: Extract from cmStandardIncludes.h 535ec2bf cmDocumentationEntry: Extract from cmStandardIncludes c3819aca cmConfigure.h: Establish as 'include first' file
| * | | cmTargetLinkLibraryType: Extract from cmStandardIncludes.hDaniel Pfeifer2016-06-162-7/+23
| | | |
| * | | cmTypeMacro: Extract from cmStandardIncludes.hDaniel Pfeifer2016-06-162-22/+38
| | | |
| * | | cmCustomCommandLines: Extract from cmStandardIncludes.hDaniel Pfeifer2016-06-162-18/+39
| | | |
| * | | cmDocumentationEntry: Extract from cmStandardIncludesDaniel Pfeifer2016-06-132-25/+46
| | | |
| * | | cmConfigure.h: Establish as 'include first' fileDaniel Pfeifer2016-06-132-12/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | At the moment, cmStandardIncludes.h needs to be included before any standard includes because it disables some warnings that are caused by the standard library of some compilers. Move this responsibility to the cmConfigure.h file. Also add include guards to cmConfigure.h to make sure the file can be included multiple times.
* | | | Merge topic 'clean-up-cmMakefile-IssueMessage'Brad King2016-06-162-6/+7
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | 5bbcf758 cmIfCommand: Don't rely on NestedError logic to issue messages 262ce91e cmMakefile: Extract invoke result variables
| * | | | cmIfCommand: Don't rely on NestedError logic to issue messagesStephen Kelly2016-06-131-3/+3
| | | | |
| * | | | cmMakefile: Extract invoke result variablesStephen Kelly2016-06-131-3/+4
| |/ / / | | | | | | | | | | | | Make it more clear what is happening here.
* | | | Merge topic 'clean-up-Parser'Brad King2016-06-163-82/+86
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | 905e738f Parser: Out-of-line conditional code to cmMakefile 0a9094cd Parser: Issue file open error messages through dedicated API
| * | | | Parser: Out-of-line conditional code to cmMakefileStephen Kelly2016-06-133-76/+77
| | | | | | | | | | | | | | | | | | | | Simplify parser API.
| * | | | Parser: Issue file open error messages through dedicated APIStephen Kelly2016-06-131-6/+9
| |/ / /
* | | | Merge topic 'find_package-duplicate-search-paths'Brad King2016-06-162-8/+26
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | 919db25c cmFindPackageCommand: remove duplicate paths from error message ebf18df5 cmFindPackageCommand: use iterators to loop over configurations
| * | | | cmFindPackageCommand: remove duplicate paths from error messageBen Boeckel2016-06-132-1/+18
| | | | | | | | | | | | | | | | | | | | Fixes #15252.
| * | | | cmFindPackageCommand: use iterators to loop over configurationsBen Boeckel2016-06-131-8/+9
| | | | |
* | | | | CMake Nightly Date StampKitware Robot2016-06-161-1/+1
| |_|_|/ |/| | |
* | | | CMake Nightly Date StampKitware Robot2016-06-151-1/+1
| |_|/ |/| |
* | | CMake Nightly Date StampKitware Robot2016-06-141-1/+1
| |/ |/|
* | Merge topic 'expose-cache-properties'Brad King2016-06-137-4/+35
|\ \ | | | | | | | | | | | | | | | | | | | | | 63c0e92c cmState: Expose list of properties of values in the cache 6eee2463 cmCacheEntry: Retrieve all properties of cache entries 120899c6 cmPropertyList: Add a way to retrieve all properties 7066218e cmake: Kill cmake::CacheManager and its getter
| * | cmState: Expose list of properties of values in the cacheTobias Hunger2016-06-102-0/+9
| | |
| * | cmCacheEntry: Retrieve all properties of cache entriesTobias Hunger2016-06-102-0/+12
| | |
| * | cmPropertyList: Add a way to retrieve all propertiesTobias Hunger2016-06-102-0/+14
| | |
| * | cmake: Kill cmake::CacheManager and its getterTobias Hunger2016-06-101-4/+0
| | | | | | | | | | | | | | | This member variable is never initialized and has apparently moved to cmState.
* | | Merge topic 'fix-cmake-ISP-violation'Brad King2016-06-135-18/+24
|\ \ \ | | | | | | | | | | | | | | | | | | | | 23f87e81 cmake: Remove force from IssueMessage API 54c65d5f cmake: Extract DisplayMessage API.
| * | | cmake: Remove force from IssueMessage APIStephen Kelly2016-06-125-19/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The force parameter is ugly and makes the method harder to reason about (issues the message ... but maybe it doesn't ... but then again you can force it). It is a violation of https://en.wikipedia.org/wiki/Interface_segregation_principle and is the kind of thing described in a recent blog here: http://code.joejag.com/2016/anti-if-the-missing-patterns.html "Any time you see this you actually have two methods bundled into one. That boolean represents an opportunity to name a concept in your code."
| * | | cmake: Extract DisplayMessage API.Stephen Kelly2016-06-122-0/+9
| | | |
* | | | Merge topic 'avoid-cmMakefile-IssueMessage-after-configure'Brad King2016-06-135-21/+49
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ea5324cd cmMakefile: Port messages for compile features to cmake df8c3130 cmGlobalGenerator: Don't use cmMakefile::IssueMessage after configure 946d1e50 cmMakefile: Avoid IssueMessage after configure is finished 096c7754 cmLocalGenerator: Store Backtrace for the directory
| * | | | cmMakefile: Port messages for compile features to cmakeStephen Kelly2016-06-122-12/+33
| | | | |
| * | | | cmGlobalGenerator: Don't use cmMakefile::IssueMessage after configureStephen Kelly2016-06-121-3/+4
| | | | |
| * | | | cmMakefile: Avoid IssueMessage after configure is finishedStephen Kelly2016-06-121-5/+9
| | | | |
| * | | | cmLocalGenerator: Store Backtrace for the directoryStephen Kelly2016-06-122-1/+3
| |/ / / | | | | | | | | | | | | Don't rely on cmMakefile to provide that. Use it to issue messages.
* | | | Merge topic 'fix-Message-API-ISP-violation'Brad King2016-06-133-52/+27
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | 0a4af073 cmake: Issue message independent of cmMakefile definition
| * | | | cmake: Issue message independent of cmMakefile definitionStephen Kelly2016-06-123-52/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The makefile is only used when called by the cmMessageCommand, so inline the use of it there. It otherwise creates an undesirable dependency on cmMakefile for issuing messages in the cmake instance, a violation of the Interface Segregation Principle. https://en.wikipedia.org/wiki/Interface_segregation_principle This also makes it more explicit that the variable definitions only affect the message() command. If an AUTHOR_WARNING is issued for any other reason, it is not affected. To affect that, it is necessary to set the cache variable instead of the regular variable. This is an unfortunate interface quirk, but one which can't be fixed easily now.
* | | | | Merge topic 'find-lib32'Brad King2016-06-135-5/+27
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 896ad251 Teach find_library and find_package to search lib32 paths (#11260)
| * | | | | Teach find_library and find_package to search lib32 paths (#11260)Daniel Scharrer2016-06-105-5/+27
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a ``FIND_LIBRARY_USE_LIB32_PATHS`` global property analogous to the ``FIND_LIBRARY_USE_LIB64_PATHS`` property. This helps find commands on multilib systems that use ``lib32`` directories and either do not have ``lib`` symlinks or point ``lib`` to ``lib64``.
* | | | | Merge topic 'add-braces'Brad King2016-06-1327-102/+195
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 757b0ff5 Add missing braces around statements in header files a16bf141 Add missing braces around statements. bd9e551c PseudoMemcheck: revise style with clang-format.
| * | | | | Add missing braces around statements in header filesDaniel Pfeifer2016-06-113-7/+14
| | | | | |
| * | | | | Add missing braces around statements.Daniel Pfeifer2016-06-1024-95/+181
| |/ / / / | | | | | | | | | | | | | | | | | | | | Apply fixits of clang-tidy's readability-braces-around-statements checker.
* | | | | Merge topic 'refactor-cmLocalGenerator-flags'Brad King2016-06-133-18/+30
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | f62ed322 cmLocalGenerator: Add GetTargetDefines to get all defines for a target 853b1bb4 cmLocalGenerator: Constify AppendDefines and AddCompileDefinitions
| * | | | | cmLocalGenerator: Add GetTargetDefines to get all defines for a targetTobias Hunger2016-06-103-11/+20
| | | | | |
| * | | | | cmLocalGenerator: Constify AppendDefines and AddCompileDefinitionsBrad King2016-06-102-7/+10
| |/ / / /
* | | | | CMake Nightly Date StampKitware Robot2016-06-131-1/+1
| |_|/ / |/| | |
* | | | CMake Nightly Date StampKitware Robot2016-06-121-1/+1
| |/ / |/| |
generated by cgit v0.12 at 2024-11-07 13:15:38 (GMT)