summaryrefslogtreecommitdiffstats
path: root/Source/cmOutputConverter.cxx
Commit message (Collapse)AuthorAgeFilesLines
* cmOutputConverter: move ConvertToRelativePath to cmStateDirectory.Bruno Manganelli2019-01-271-12/+0
|
* cmOutputConverter: Moved ContainedInDirectory to cmStateDirectoryBruno Manganelli2018-12-081-33/+2
|
* cmOutputConverter: Moved ForceToRelativePath to cmSystemBruno Manganelli2018-12-071-75/+1
|
* clang-tidy: Fix readability-static-accessed-through-instanceRegina Pfeifer2018-11-211-1/+1
| | | | Enable the check in .clang-tidy and fix all warnings.
* Using front() and back() instead of calculationsCengizhan Pasaoglu2018-11-061-3/+3
|
* cmStateDirectory: use const std::string& for return valuesVitaly Stakhovsky2018-08-121-2/+2
|
* cmOutputConverter: mark helper functions staticRolf Eike Beer2018-03-061-6/+4
| | | | This allows the linker to entirely drop any symbols of the functions.
* cmOutputConverter: Inline some functions and avoid string re-allocationsFrank Winklmeier2018-02-231-32/+37
| | | | | | | Use std::string (with correct initial size) in cmOutputConverter::Shell__GetArgument instead of ostringstream. This avoids several re-allocations of the string buffer. In addition, convert some of the private static members into inline free functions to avoid function calls.
* Reduce raw string pointers usage.Pavel Solodovnikov2018-01-311-2/+2
| | | | | | | | | | * Change some functions to take `std::string` instead of `const char*` in the following classes: `cmMakeFile`, `cmake`, `cmCoreTryCompile`, `cmSystemTools`, `cmState`, `cmLocalGenerator` and a few others. * Greatly reduce using of `const char*` overloads for `cmSystemTools::MakeDirectory` and `cmSystemTools::RelativePath`. * Remove many redundant `c_str()` conversions throughout the code.
* 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.