summaryrefslogtreecommitdiffstats
path: root/Source/cmSourceGroupCommand.cxx
Commit message (Collapse)AuthorAgeFilesLines
* source_group: ensure that passed file is not a directoryMateusz Janek2019-11-041-3/+11
| | | | Fixes: #19769
* cmSourceGroupCommand: Port away from cmCommandRegina Pfeifer2019-09-211-34/+46
|
* clang-tidy: modernize-deprecated-headersRegina Pfeifer2019-09-161-1/+1
|
* clang-tidy: modernize-use-autoRegina Pfeifer2019-09-101-2/+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-6/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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)`
* cmAlgorithms: Add cmContainsRegina Pfeifer2019-08-191-3/+2
| | | | Also, use the new function where applicable.
* Merge topic 'tidy_inefficient_string'Brad King2019-08-071-2/+2
|\ | | | | | | | | | | | | 18b0330b86 clang-tidy: Enable performance-inefficient-string-concatenation Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !3648
| * clang-tidy: Enable performance-inefficient-string-concatenationSebastian Holtermann2019-08-051-2/+2
| | | | | | | | | | | | | | Enables the clang-tidy test performance-inefficient-string-concatenation and replaces all inefficient string concatenations with `cmStrCat`. Closes: #19555
* | cmStringAlgorithms: cmIsSpace, cmTrimWhitespace, cmEscapeQuotes, cmTokenizeSebastian Holtermann2019-08-051-1/+1
|/ | | | | | | | | | This adds the following functions to `cmStringAlgorithms`: - `cmIsSpace` - `cmTrimWhitespace` (moved from `cmSystemTools::TrimWhitespace`) - `cmEscapeQuotes` (moved from `cmSystemTools::EscapeQuotes`) - `cmTokenize` (moved from `cmSystemTools::tokenize` and adapted to accept `cm::string_view`)
* Cleanups: Use cmHas{Prefix,Suffix} instead of String{Starts,Ends}WithSebastian Holtermann2019-08-011-1/+2
|
* source_group: Fix regression in relative FILESBrad King2019-07-301-11/+4
| | | | | | | | | | Fix the check added in commit 8d93815d20 (source_group command ensures that FILES arguments are actually files, 2019-04-25, v3.15.0-rc1~195^2) to convert to an absolute path before checking for existence. Also simplify the conversion to an absolute path. Fixes: #19454
* source_group command ensures that FILES arguments are actually filesMateusz Janek2019-04-251-1/+4
|
* Enabled performance-inefficient-vector-operation check in clang-tidyJakub Kalinski2019-04-021-0/+1
|
* clang-tidy: Use emplaceRegina Pfeifer2019-01-171-1/+1
|
* clang-tidy: Simplify boolean expressionsRegina Pfeifer2019-01-151-16/+8
|
* Source_Group: Allow an empty prefix with the TREE syntax.Yohann Benedic2018-04-301-1/+5
| | | | | | | | | | | | When calling source_group(TREE <root> PREFIX "" FILES ...) the files located at the root directory were not assigned to the correct source group: they were put in a default group (e.g. "Source Files" in Visual Studio). Fixes: #17608
* 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.
* source_group: Fix TREE argument parsingMateusz Janek2018-01-301-83/+141
| | | | Fixes: #17581
* Merge topic 'ranged-for'Brad King2017-09-131-11/+10
|\ | | | | | | | | | | | | 7d509579 Meta: modernize old-fashioned loops to range-based `for`. Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1249
| * Meta: modernize old-fashioned loops to range-based `for`.Pavel Solodovnikov2017-09-121-11/+10
| | | | | | | | | | | | Changes done via `clang-tidy` with some manual fine-tuning for the variable naming and `auto` type deduction where appropriate.
* | Use cmMakefile::GetOrCreateSourceGroup in cmSourceGroupCommandSebastian Holtermann2017-09-091-32/+2
|/
* Use C++11 nullptrDaniel Pfeifer2017-08-241-2/+2
|
* Merge topic 'source_group-TREE-relative-path'Brad King2017-05-181-12/+20
|\ | | | | | | | | | | | | 4716f2be source_group: Restore TREE support for relative paths Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !852
| * source_group: Restore TREE support for relative pathsMateusz Janek2017-05-171-12/+20
| | | | | | | | | | | | | | | | | | | | | | The fix in commit v3.8.1~4^2 (source_group: Fix TREE with root that is not current source dir, 2017-04-20) accidentally broke support for specifying paths relative to the source directory. Fix it and add a test covering the case. While at it, fix a typo in a variable name. Fixes: #16876
* | Merge branch 'backport-source_group-TREE-alt-root' into ↵Brad King2017-04-211-16/+43
|\ \ | |/ | | | | source_group-TREE-alt-root
| * source_group: Fix TREE with root that is not current source dirMateusz Janek2017-04-211-16/+44
| | | | | | | | | | | | Also raise an error if files are given that are not below the root. Fixes: #16807
* | fix some include-what-you-use diagnosticsDaniel Pfeifer2017-02-171-0/+2
|/
* source_group: Add options create groups matching directory treeMateusz Janek2017-01-161-0/+165
| | | | Add `TREE` and `PREFIX` arguments to enable this behavior.
* Remove redundant c_str() callsDaniel Pfeifer2016-10-301-1/+1
|
* Separate compilation for commands included in cmCommandsDaniel Pfeifer2016-10-211-0/+8
|
* 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
|
* use CM_NULLPTRDaniel Pfeifer2016-06-281-1/+1
|
* Revise C++ coding style using clang-formatKitware Robot2016-05-161-40/+23
| | | | | | | | | | | | | 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: Rename GetCurrent{,Source}Directory.Stephen Kelly2015-04-201-1/+1
| | | | Match the names used in cmake code.
* Port all cmOStringStream to std::ostringstream.Stephen Kelly2015-01-111-1/+1
| | | | All compilers hosting CMake support the std class.
* Remove c_str calls when using stream APIs.Stephen Kelly2014-03-111-1/+1
| | | | | Use an ad-hoc clang tool for matching the calls which should be ported.
* Remove some c_str() calls.Stephen Kelly2014-03-111-3/+3
| | | | | | Use the clang RemoveCStrCalls tool to automatically migrate the code. This was only run on linux, so does not have any positive or negative effect on other platforms.
* Remove trailing whitespace from most CMake and C/C++ codeKitware Robot2012-08-131-4/+4
| | | | | | | | | | | | | | | | | 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 style errors added by parent and grandparentBrad King2011-06-081-1/+2
|
* CMake: Move tokenize to cmSystemToolsJohan Björk2011-06-071-32/+1
|
* 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.
* ENH: add return and break support to cmake, also change basic command ↵Ken Martin2008-01-231-1/+2
| | | | invocation signature to be able to return extra informaiton via the cmExecutionStatus class
* COMP: parent is not used anymore with this patch, since now the name isAlexander Neundorf2007-08-261-2/+1
| | | | | | given as a vector of components Alex
* BUG: fix #4057 (which had several duplicates): handle recursivew source ↵Alexander Neundorf2007-08-241-8/+5
| | | | | | groups better, i.e. multiple sourcegroups with the same end component work now Alex
* BUG: handle source_group names which consist only of the delimiter the same ↵Alexander Neundorf2007-08-241-8/+7
| | | | | | was as empty source group names Alex
* STYLE: some m_ to this-> cleanupKen Martin2006-03-151-6/+6
|
* ENH: fix problem if there are .. in the path to the source file specified in ↵Bill Hoffman2006-02-281-0/+1
| | | | a source group
* ENH: fix warning and remove unused variableBill Hoffman2006-02-241-3/+2
|
* ENH: fix for bug 2908 crash for empty source group nameBill Hoffman2006-02-231-22/+14
|