summaryrefslogtreecommitdiff
path: root/Source/cmIncludeDirectoryCommand.cxx
Commit message (Collapse)AuthorAgeFilesLines
* Refactoring: use append functions from cmext/algorithmMarc Chevrier2019-12-171-3/+4
|
* cmIncludeDirectoryCommand: Port away from cmCommandRegina Pfeifer2019-09-211-22/+22
|
* clang-tidy: modernize-use-autoRegina Pfeifer2019-09-101-1/+1
| | | | | | Set the MinTypeNameLength option to an impossibly high value in order to limit the diagnostics to iterators. Leave new expressions and cast expressions for later.
* Source sweep: Use cmStrCat for string concatenationSebastian Holtermann2019-08-221-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch is generated by a python script that uses regular expressions to search for string concatenation patterns of the kind ``` std::string str = <ARG0>; str += <ARG1>; str += <ARG2>; ... ``` and replaces them with a single `cmStrCat` call ``` std::string str = cmStrCat(<ARG0>, <ARG1>, <ARG2>, ...); ``` If any `<ARGX>` is itself a concatenated string of the kind ``` a + b + c + ...; ``` then `<ARGX>` is split into multiple arguments for the `cmStrCat` call. If there's a sequence of literals in the `<ARGX>`, then all literals in the sequence are concatenated and merged into a single literal argument for the `cmStrCat` call. Single character strings are converted to single char arguments for the `cmStrCat` call. `std::to_string(...)` wrappings are removed from `cmStrCat` arguments, because it supports numeric types as well as string types. `arg.substr(x)` arguments to `cmStrCat` are replaced with `cm::string_view(arg).substr(x)`
* Source sweep: Use cmIsOn instead of cmSystemTools::IsOnSebastian Holtermann2019-08-171-1/+2
| | | | | | | | | This replaces invocations of - `cmSystemTools::IsInternallyOn` with `cmIsInternallyOn` - `cmSystemTools::IsNOTFOUND` with `cmIsNOTFOUND` - `cmSystemTools::IsOn` with `cmIsOn` - `cmSystemTools::IsOff` with `cmIsOff`
* Use cmAppend to append ranges to std::vector instancesSebastian Holtermann2019-05-231-4/+3
|
* IWYU: Update CMake code for IWYU built with Clang 6Brad King2019-01-151-0/+1
| | | | | IWYU now correctly requires `<utility>` for `std::move`. It also requires a container header when used via a range-based for loop.
* Refactoring: introduce function to check if a string is a generator expressionMarc Chevrier2018-09-251-6/+2
|
* Remove unnecessary c_str() callsVitaly Stakhovsky2018-09-051-1/+1
| | | | Use the new IsOn(),IsOff() overloads.
* Reduce raw string pointers usage.Pavel Solodovnikov2018-01-311-1/+1
| | | | | | | | | | * 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.
* Reduce allocation of temporary values on heap.Pavel Solodovnikov2018-01-261-2/+2
| | | | | - Use `std::move` while inserting temporary results into vectors. - Change `push_back` to `emplace_back` where appropriate.
* Fix trivial typos in textluzpaz2017-11-031-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | Some are user facing. Found using codespell -q 3 --skip="./Utilities" -I .cmake-whitelist.txt` whereby the whitelist contained: ans dum helpfull emmited emmitted buil iff isnt nto ot pathes substract te todays upto whitespaces
* Meta: replace empty-string assignments with `clear()`.Pavel Solodovnikov2017-09-161-1/+1
|
* Access string npos without instancePavel Solodovnikov2017-06-011-1/+1
|
* Include necessary headers in commandsDaniel Pfeifer2016-10-261-0/+6
|
* Separate compilation for commands included in cmBootstrapCommands2Daniel Pfeifer2016-10-211-0/+2
|
* 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.
* use empty method to check for emptynessDaniel Pfeifer2016-09-151-1/+1
|
* Revise C++ coding style using clang-formatKitware Robot2016-05-161-62/+39
| | | | | | | | | | | | | 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.
* cmMakefile: Port users of GetStart* methods to new names.Stephen Kelly2015-04-211-1/+1
|
* Replace 'foo.size() == 0' pattern with foo.empty().Stephen Kelly2015-01-181-1/+1
|
* Use two-iterator std::set::insert where appropriate.Stephen Kelly2015-01-111-5/+1
|
* strings: Remove cmStdString referencesBen Boeckel2014-03-081-1/+1
| | | | | | | | | | | Casts from std::string -> cmStdString were high on the list of things taking up time. Avoid such implicit casts across function calls by just using std::string everywhere. The comment that the symbol name is too long is no longer relevant since modern debuggers alias the templates anyways and the size is a non-issue since the underlying methods are generated since it's inherited.
* Style: Use this-> when invoking member function.Stephen Kelly2014-01-041-1/+1
|
* include_directories: Fix handling of empty or space-only entriesStephen Kelly2013-05-211-2/+13
| | | | | | | | | This is a regression introduced in commit 0d46e9a0 (Store includes from the same include_directories call together., 2013-01-20). Reported at http://thread.gmane.org/gmane.comp.programming.tools.cmake.user/46695/focus=7847
* Style: Use this-> when invoking member functions.Stephen Kelly2013-02-071-2/+2
|
* Store includes from the same include_directories call together.Stephen Kelly2013-01-211-34/+58
| | | | | | Otherwise, we get a separate IncludeDirectoriesEntry for each include, and that causes unnecessary and confusing splitting in the output when debugging the INCLUDE_DIRECTORIES property.
* Process generator expressions in the INCLUDE_DIRECTORIES property.Stephen Kelly2012-09-281-1/+6
| | | | | This use of generator expressions, like all others to come which operate on target properties, must initalize the dag checker.
* Don't prepend a path before generator expressions in include_directories.Stephen Kelly2012-09-211-4/+7
|
* Remove trailing whitespace from most CMake and C/C++ codeKitware Robot2012-08-131-3/+3
| | | | | | | | | | | | | | | | | Our Git commit hooks disallow modification or addition of lines with trailing whitespace. Wipe out all remnants of trailing whitespace everywhere except third-party code. Run the following shell code: git ls-files -z -- \ bootstrap doxygen.config '*.readme' \ '*.c' '*.cmake' '*.cpp' '*.cxx' \ '*.el' '*.f' '*.f90' '*.h' '*.in' '*.in.l' '*.java' \ '*.mm' '*.pike' '*.py' '*.txt' '*.vim' | egrep -z -v '^(Utilities/cm|Source/(kwsys|CursesDialog/form)/)' | egrep -z -v '^(Modules/CPack\..*\.in)' | xargs -0 sed -i 's/ \+$//'
* Fix spelling errors reported by Lintian.Kai Wasserbäch2010-07-131-1/+1
| | | | | During a Lintian run on the binary packages of CMake in Debian I was notified of many spelling mistakes.
* Convert CMake to OSI-approved BSD LicenseBrad King2009-09-281-14/+9
| | | | | | | This converts the CMake license to a pure 3-clause OSI-approved BSD License. We drop the previous license clause requiring modified versions to be plainly marked. We also update the CMake copyright to cover the full development time range.
* BUG: fix #8704, sometimes crash if include_directories() is called with a ↵Alexander Neundorf2009-03-121-12/+7
| | | | | | whitespace string Alex
* BUG: Fix include_directories command to produce an immediately whether or ↵Brad King2008-03-081-11/+2
| | | | not 2.4 compatibility is enabled. CMake 2.4 already produced an error, just not immediately.
* BUG: Fix crash when CMAKE_BACKWARDS_COMPATIBILITY is not set.Brad King2008-03-041-3/+1
|
* ENH: add first cut and policies still need to add the doc supportKen Martin2008-03-011-1/+1
|
* ENH: add return and break support to cmake, also change basic command ↵Ken Martin2008-01-231-1/+1
| | | | invocation signature to be able to return extra informaiton via the cmExecutionStatus class
* STYLE: Fix line-too-long.Brad King2007-03-081-1/+2
|
* BUG: improve bad argument handling for INCLUDE_DIRECTORIES and ↵Ken Martin2007-03-071-14/+67
| | | | ADD_DEFINITIONS bug 4364
* BUG: Produce error when include directories is invoked with an empty listAndy Cedilnik2007-02-231-2/+13
|
* ENH: Adding SYSTEM option to INCLUDE_DIRECTORIES command. This addresses ↵Brad King2006-10-051-0/+10
| | | | bug #3462.
* ENH: fix bad error reporting with not found pathsKen Martin2006-08-231-14/+9
|
* COMP: Handle both ansi and non-ansi CAndy Cedilnik2006-07-261-0/+8
|
* BUG: INCLUDE_DIRECTORIES should interpret relative path arguments with ↵Brad King2006-05-121-0/+7
| | | | respect to the current source directory.
* STYLE: fix line lengthKen Martin2006-05-111-2/+4
|
* ENH: INCLUDE_DIRECTORIES should have been written to prepend to the include ↵Brad King2006-04-041-1/+7
| | | | path so that the most local directories are included first. This is a patch from Alex to resolve the problem by allowing users to switch the default using a variable CMAKE_INCLUDE_DIRECTORIES_BEFORE and then still explicitly appending or prepending by using AFTER or BEFORE arguments explicitly.
* ENH: make sure include and lib dirs are unix pathsBill Hoffman2006-04-031-1/+3
|
* STYLE: some m_ to this-> cleanupKen Martin2006-03-151-1/+1
|
* ENH: add a check for empty include directoriesBill Hoffman2004-10-281-0/+4
|
* ENH: add checking for NOTFOUNDBill Hoffman2003-01-311-5/+0
|