summaryrefslogtreecommitdiffstats
path: root/Source/cmOutputConverter.cxx
Commit message (Collapse)AuthorAgeFilesLines
* Meta: modernize old-fashioned loops to range-based `for`.Pavel Solodovnikov2017-09-121-4/+3
| | | | | | Changes done via `clang-tidy` with some manual fine-tuning for the variable naming and `auto` type deduction where appropriate.
* Replace C-style castsDaniel Pfeifer2017-08-271-1/+1
|
* Pass large types by const&, small types by valueDaniel Pfeifer2017-06-031-2/+2
|
* Fix several include-what-you-use findingsDaniel Pfeifer2016-11-081-5/+6
|
* cmState: Split auxiliary classes into separate filesStephen Kelly2016-10-191-0/+2
| | | | | | | Port dependents to the new locations as needed. Leave behind a cmState.h include in cmListFileCache to reduce noise. It is removed in a following commit.
* cmState: Port dependent code to new cmStateSnapshot nameStephen Kelly2016-10-191-1/+1
|
* cmState: Port dependent code to new cmStateDirectory nameStephen Kelly2016-10-191-1/+1
|
* cmOutputConverter: Add a flag for IsUnixStephen Kelly2016-10-061-16/+16
| | | | Remove the need for method parameters to represent the distinction.
* Convert: Inline platform-specific methodsStephen Kelly2016-10-061-16/+3
| | | | They don't provide real value.
* Convert: Extract method to determine if paths are in directoryStephen Kelly2016-10-061-5/+13
| | | | | | The conditional early return can be moved to clients, which would have many benefits, notably making cmOutputConverter independent of directory-specific state.
* Convert: Remove asserts which are duplicated in delegate methodStephen Kelly2016-10-061-7/+0
| | | | | This means that we don't encounter the asserts in the case where we early-return from here.
* Convert: Remove early return checkStephen Kelly2016-10-061-5/+0
| | | | This function delegates to another function which does the same check.
* Convert: Extract local variables for readabilityStephen Kelly2016-10-061-12/+13
|
* Convert: Extract local variablesStephen Kelly2016-10-061-14/+17
| | | | Remove comment made obsolete by them.
* cmOutputConverter: remove unused codeDaniel Pfeifer2016-10-061-9/+0
| | | | | Remove old ConvertToRelativePath function now that all clients have migrated to the new signature.
* Simplify CMake per-source license noticesBrad King2016-09-271-11/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | Per-source copyright/license notice headers that spell out copyright holder names and years are hard to maintain and often out-of-date or plain wrong. Precise contributor information is already maintained automatically by the version control tool. Ultimately it is the receiver of a file who is responsible for determining its licensing status, and per-source notices are merely a convenience. Therefore it is simpler and more accurate for each source to have a generic notice of the license name and references to more detailed information on copyright holders and full license terms. Our `Copyright.txt` file now contains a list of Contributors whose names appeared source-level copyright notices. It also references version control history for more precise information. Therefore we no longer need to spell out the list of Contributors in each source file notice. Replace CMake per-source copyright/license notice headers with a short description of the license and links to `Copyright.txt` and online information available from "https://cmake.org/licensing". The online URL also handles cases of modules being copied out of our source into other projects, so we can drop our notices about replacing links with full license text. Run the `Utilities/Scripts/filter-notices.bash` script to perform the majority of the replacements mechanically. Manually fix up shebang lines and trailing newlines in a few files. Manually update the notices in a few files that the script does not handle.
* cmOutputConverter: Remove now-obsolete Convert methodStephen Kelly2016-09-191-23/+0
|
* Convert: Remove HOME_OUTPUT enum valueStephen Kelly2016-09-191-3/+0
|
* Convert: Simplify switchStephen Kelly2016-09-191-5/+4
| | | | Make it more clear what is happening here.
* Convert: Remove obsolete MAKERULE enum valueStephen Kelly2016-09-191-3/+1
|
* Convert: Remove obsolete HOME and START enum valuesStephen Kelly2016-09-191-8/+0
|
* cmOutputConverter: use new ConvertToRelativePath signature internallyDaniel Pfeifer2016-09-171-6/+4
|
* cmOutputConverter: split ConvertToRelativePathDaniel Pfeifer2016-09-171-27/+53
| | | | | | | Split the ConvertToRelativePath funtion into ConvertToRelativePath and ForceToRelativePath. Both functions take the local path as a string rather than a vector of path segments. Reimplement the old interface on top of the two new functions for interface compatibility.
* fix a load of include-what-you-use violationsDaniel Pfeifer2016-09-031-4/+4
|
* Convert: Make variables a bit more clearStephen Kelly2016-08-271-5/+5
|
* Convert: Remove 'FULL' conversionStephen Kelly2016-08-271-3/+0
| | | | It is no longer used.
* Convert: Extract ConvertToRelativePath from Convert()Stephen Kelly2016-08-271-4/+11
| | | | | | | Convert() does some kind of relative conversion, followed by a conversion to 'output format'. Make it possible to do the former without the latter.
* Convert: Remove NONE conversionStephen Kelly2016-08-271-2/+0
| | | | It is no longer used.
* Convert: Remove obsolete GetRelativeRootPathStephen Kelly2016-08-271-17/+0
|
* Convert: Remove ConvertToOutputForExisting overloadStephen Kelly2016-08-271-13/+0
| | | | It is no longer used.
* Convert: Remove unused overloadStephen Kelly2016-08-271-21/+0
|
* use CM_NULLPTRDaniel Pfeifer2016-06-281-3/+3
|
* cmOutputConverter: remove unnecessary conversionDaniel Pfeifer2016-06-201-6/+4
|
* cmOutputConverter: collapse ConvertToOutputForExisting functionsDaniel Pfeifer2016-06-201-18/+6
|
* cmOutputConverter: remove unused 'local' argumentDaniel Pfeifer2016-06-201-6/+2
|
* cmOutputConverter: remove 'optional' argumentDaniel Pfeifer2016-06-201-5/+12
| | | | | | Remove the 'optional' paramenter from the second overload of the Convert function. This parameter is used from one single location. Inline the codepath for which the argument is true to the callsite.
* cmOutputConverter::Convert: invert conditionDaniel Pfeifer2016-06-201-6/+6
| | | | Make the control flow of the 'optional' argument more explicit.
* cmOutputConverter::Convert: make precondition explicitDaniel Pfeifer2016-06-201-2/+4
|
* cmOutputConverter: implement Shell__GetArgument using ostringstreamDaniel Pfeifer2016-06-151-178/+36
| | | | | This removes the need to calculate the resulting string length beforehand.
* Revise C++ coding style using clang-formatKitware Robot2016-05-161-440/+279
| | | | | | | | | | | | | Run the `Utilities/Scripts/clang-format.bash` script to update all our C++ code to a new style defined by `.clang-format`. Use `clang-format` version 3.8. * If you reached this commit for a line in `git blame`, re-run the blame operation starting at the parent of this commit to see older history for the content. * See the parent commit for instructions to rebase a change across this style transition commit.
* Remove `//------...` horizontal separator commentsBrad King2016-05-091-28/+0
| | | | | | | | | | | | | | | | | | | | | | | | Modern editors provide plenty of ways to visually separate functions. Drop the explicit comments that previously served this purpose. Use the following command to automate the change: $ git ls-files -z -- \ "*.c" "*.cc" "*.cpp" "*.cxx" "*.h" "*.hh" "*.hpp" "*.hxx" | egrep -z -v "^Source/cmCommandArgumentLexer\." | egrep -z -v "^Source/cmCommandArgumentParser(\.y|\.cxx|Tokens\.h)" | egrep -z -v "^Source/cmDependsJavaLexer\." | egrep -z -v "^Source/cmDependsJavaParser(\.y|\.cxx|Tokens\.h)" | egrep -z -v "^Source/cmExprLexer\." | egrep -z -v "^Source/cmExprParser(\.y|\.cxx|Tokens\.h)" | egrep -z -v "^Source/cmFortranLexer\." | egrep -z -v "^Source/cmFortranParser(\.y|\.cxx|Tokens\.h)" | egrep -z -v "^Source/cmListFileLexer\." | egrep -z -v "^Source/cm_sha2" | egrep -z -v "^Source/(kwsys|CursesDialog/form)/" | egrep -z -v "^Utilities/(KW|cm).*/" | xargs -0 sed -i '/^\(\/\/---*\|\/\*---*\*\/\)$/ {d;}' This avoids modifying third-party sources and generated sources.
* Format include directive blocks and ordering with clang-formatBrad King2016-04-291-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Sort include directives within each block (separated by a blank line) in lexicographic order (except to prioritize `sys/types.h` first). First run `clang-format` with the config file: --- SortIncludes: false ... Commit the result temporarily. Then run `clang-format` again with: --- SortIncludes: true IncludeCategories: - Regex: 'sys/types.h' Priority: -1 ... Commit the result temporarily. Start a new branch and cherry-pick the second commit. Manually resolve conflicts to preserve indentation of re-ordered includes. This cleans up the include ordering without changing any other style. Use the following command to run `clang-format`: $ git ls-files -z -- \ '*.c' '*.cc' '*.cpp' '*.cxx' '*.h' '*.hh' '*.hpp' '*.hxx' | egrep -z -v '(Lexer|Parser|ParserHelper)\.' | egrep -z -v '^Source/cm_sha2' | egrep -z -v '^Source/(kwsys|CursesDialog/form)/' | egrep -z -v '^Utilities/(KW|cm).*/' | egrep -z -v '^Tests/Module/GenerateExportHeader' | egrep -z -v '^Tests/RunCMake/CommandLine/cmake_depends/test_UTF-16LE.h' | xargs -0 clang-format -i This selects source files that do not come from a third-party. Inspired-by: Daniel Pfeifer <daniel@pfeifer-mail.de>
* cmOutputConverter: Assert construction with a valid snapshotBrad King2016-04-151-0/+1
| | | | | We unconditionally use information from the snapshot so it must be valid.
* Genex: Add a SHELL_PATH expressionStefan Kislinskiy2015-09-281-15/+24
| | | | | | | | | Some commands on Windows do not understand forward slash paths and require backslashes. In order to help projects generate shell invocations of such commands, provide a generator expression to convert paths to the shell-preferred path format for the current generator. This will allow custom commands to generate paths the same way CMake does for compiler command invocations.
* cmAlgorithms: Rename cmRange to cmMakeRange.Stephen Kelly2015-07-221-1/+1
|
* cmState: Extract a Directory class.Stephen Kelly2015-07-181-14/+20
| | | | | Move Directory-scoped state accessors to it. This will be expanded with directory property state soon.
* cmOutputConverter: Adopt command line escaping codeBrad King2015-07-081-16/+599
| | | | | | | Port code from the KWSys System_Shell APIs into cmOutputConverter. Drop it from our copy of KWSys because upstream will drop it too, and by doing it in this commit 'git blame' may have an easier time connecting the history of the content.
* cmOutputConverter: Adopt EscapeWindowsShellArgument methodBrad King2015-07-081-0/+20
| | | | Move it out of cmSystemTools and into cmOutputConverter.
* cmOutputConverter: Constify API.Stephen Kelly2015-06-211-9/+9
|
* cmOutputConverter: Extract from cmLocalGenerator.Stephen Kelly2015-06-061-0/+447
The Convert methods never belonged to the local generator concept, so split them out now. The cmOutputConverter is cheap to construct and destroy, so it can be instantiated where needed to perform conversions. This will allow further decoupling of cmLocalGenerator from the configure step. Inherit cmLocalGenerator from cmOutputConverter for the purpose of source compatibility.