summaryrefslogtreecommitdiffstats
path: root/Source
Commit message (Collapse)AuthorAgeFilesLines
* Remove `//------...` horizontal separator commentsBrad King2016-05-09217-2623/+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.
* Merge topic 'custom-command-CROSSCOMPILING_EMULATOR'Brad King2016-05-092-2/+33
|\ | | | | | | | | 8c2cedc6 CustomCommandGenerator: Add support for CROSSCOMPILING_EMULATOR
| * CustomCommandGenerator: Add support for CROSSCOMPILING_EMULATORJean-Christophe Fillion-Robin2016-05-092-2/+33
| | | | | | | | | | | | Teach the `add_custom_command` and `add_custom_target' commands to substitute argv0 with the crosscompiling emulator if it is a target with the `CROSSCOMPILING_EMULATOR` property set.
* | Merge topic 'clang-format-prep'Brad King2016-05-0954-13/+314
|\ \ | | | | | | | | | | | | | | | | | | afca3735 Help clang-format wrap after braces on long initializer lists 85425a3e Move comments off of class access specifier lines 64b55203 Isolate formatted streaming blocks with clang-format off/on
| * | Help clang-format wrap after braces on long initializer listsBrad King2016-05-064-0/+4
| | | | | | | | | | | | | | | | | | Add a long comment inside a few braced initializer lists in order to convince clang-format to break after the opening brace and format the list without indenting every value past the opening brace.
| * | Move comments off of class access specifier linesBrad King2016-05-065-13/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The clang-format tool may turn this: public: // comment about access specifier // unrelated comment indented with code ... Into: public: // comment about access specifier // unrelated comment indented with code ... Avoid this by moving comments off of access specifier lines.
| * | Isolate formatted streaming blocks with clang-format off/onBrad King2016-05-0646-0/+282
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The clang-format tool can do a good job formatting most code, but well-organized streaming blocks are best left manually formatted. Find blocks of the form os << "...\n" "...\n" ; using the command $ git ls-files -z -- Source | egrep -v -z '^Source/kwsys/' | xargs -0 pcregrep -M --color=always -B 1 -A 1 -n \ '<<[^\n]*\n(^ *("[^\n]*("|<<|;)$|;)\n){2,}' Find blocks of the form os << "...\n" << "...\n" << "...\n"; using the command $ git ls-files -z -- Source | egrep -v -z '^Source/kwsys/' | xargs -0 pcregrep -M --color=always -B 1 -A 1 -n \ '<<[^\n]*\n(^ *<<[^\n]*(\\n"|<<|;)$\n){2,}' Surround such blocks with the pair /* clang-format off */ ... /* clang-format on */ in order to protect them from update by clang-format. Use the C-style `/*...*/` comments instead of C++-style `//...` comments in order to prevent them from ever being swallowed by re-formatting of surrounding comments.
* | | Merge topic 'fix-export-unset'Brad King2016-05-091-0/+3
|\ \ \ | | | | | | | | | | | | | | | | f45c16e7 Fix export file variable pollution
| * | | Fix export file variable pollutionMatthew Woehlke2016-05-051-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Add missing unset of scratch variables to generated export file for case that the file was already included. We already unset these when parsing the file normally, but the multiple inclusion case was leaving them around.
* | | | CMake Nightly Date StampKitware Robot2016-05-091-1/+1
| | | |
* | | | CMake Nightly Date StampKitware Robot2016-05-081-1/+1
| | | |
* | | | CMake Nightly Date StampKitware Robot2016-05-071-1/+1
| |/ / |/| |
* | | Merge topic 'ghs-hash-object-locations'Brad King2016-05-064-24/+85
|\ \ \ | | | | | | | | | | | | | | | | 5e428713 GHS: Shorten long object paths with duplicate source names
| * | | GHS: Shorten long object paths with duplicate source namesGeoff Viola2016-05-064-24/+85
| |/ / | | | | | | | | | | | | Detect when the resulting object path is too long and compute an alternative name using a hash.
* | | CMake Nightly Date StampKitware Robot2016-05-061-1/+1
|/ /
* | CMake Nightly Date StampKitware Robot2016-05-051-1/+1
| |
* | CMake Nightly Date StampKitware Robot2016-05-041-1/+1
|/
* Merge topic 'doc-cmake-E-updates'Brad King2016-05-031-1/+1
|\ | | | | | | | | | | 960afaad Help: Improve 'cmake -E md5sum' documentation 25ee2c86 Help: Add missing space in cmake(1) manual
| * Help: Improve 'cmake -E md5sum' documentationBartosz Kosiorek2016-05-021-1/+1
| |
* | Merge topic 'doc-cmake-policies-manual-improvements'Brad King2016-05-033-4/+15
|\ \ | | | | | | | | | | | | | | | | | | | | | 5f948d2a Help: Add policy summaries to cmake-policies(7) b74d73e5 Help: Organize cmake-policies(7) manual by version of introduction de370656 cmRST: Parse toctree lines with Sphinx cross-reference syntax 845cb217 CMP0059: Fix typo in policy description
| * | cmRST: Parse toctree lines with Sphinx cross-reference syntaxBrad King2016-05-022-3/+14
| | |
| * | CMP0059: Fix typo in policy descriptionBrad King2016-05-021-1/+1
| |/
* | Merge topic 'clang-format-include-order'Brad King2016-05-0315-13/+22
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | 54f71cd7 Source: Sort includes the way clang-format would 7b6ffa59 Source: Sort includes of sys/types.h as clang-format would be14fe48 Source: Stabilize include order of sys/types.h before sys/stat.h 5e871f70 Tests: Sort includes of sys/types.h as clang-format would a20d7d48 Tests: Fix Plugin test include order
| * | Source: Sort includes the way clang-format wouldBrad King2016-05-034-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Re-apply the approach from commit e1c77472 (Format include directive blocks and ordering with clang-format, 2016-04-29) but this time be more careful about exclusion of parser generator sources: $ git ls-files -z -- \ '*.c' '*.cc' '*.cpp' '*.cxx' '*.h' '*.hh' '*.hpp' '*.hxx' | egrep -z -v '^Source/cmCommandArgumentLexer\.' | egrep -z -v '^Source/cmCommandArgumentParser\.' | egrep -z -v '^Source/cmDependsJavaLexer\.' | egrep -z -v '^Source/cmDependsJavaParser\.' | egrep -z -v '^Source/cmExprLexer\.' | egrep -z -v '^Source/cmExprParser\.' | egrep -z -v '^Source/cmFortranLexer\.' | egrep -z -v '^Source/cmFortranParser\.' | egrep -z -v '^Source/cmListFileLexer\.' | 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 Also drop use of custom sorting for `sys/types.h`.
| * | Source: Sort includes of sys/types.h as clang-format wouldBrad King2016-05-034-4/+4
| | | | | | | | | | | | | | | When this header is included in blocks where order does not matter, just place it in lexicographic order as clang-format would by default.
| * | Source: Stabilize include order of sys/types.h before sys/stat.hBrad King2016-05-037-4/+13
| | | | | | | | | | | | | | | Include the two headers in an isolated block with a comment separating them so that tools that re-order includes do not re-order these.
* | | CMake Nightly Date StampKitware Robot2016-05-031-1/+1
| |/ |/|
* | Merge topic 'clang-format-include-order'Brad King2016-05-02348-769/+978
|\ \ | |/ | | | | | | | | | | | | | | | | | | e1c77472 Format include directive blocks and ordering with clang-format 180538c7 Source: Stabilize include order 0e7bca92 Utilities/Release: Stabilize include order in WiX custom action eb817be0 Tests: Stabilize include order in MFC, VSXaml, and VSWinStorePhone eda313b4 Tests: Stabilize include order in StringFileTest 7110b754 CursesDialog: add missing cmState include d7a5f255 Modules: Remove unused CMakeTestWatcomVersion.c file
| * Format include directive blocks and ordering with clang-formatBrad King2016-04-29214-661/+661
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
| * Source: Stabilize include orderBrad King2016-04-29254-114/+322
| | | | | | | | | | Each source file has a logical first include file. Include it in an isolated block so that tools that sort includes do not move them.
| * CursesDialog: add missing cmState includeDaniel Pfeifer2016-04-291-0/+1
| |
* | CMake Nightly Date StampKitware Robot2016-05-021-1/+1
| |
* | CMake Nightly Date StampKitware Robot2016-05-011-1/+1
| |
* | CMake Nightly Date StampKitware Robot2016-04-301-1/+1
| |
* | Merge topic 'vs71-deprecate'Brad King2016-04-292-1/+18
|\ \ | | | | | | | | | | | | | | | ce82e0a5 Deprecate Visual Studio 7 .NET 2003 generator 6a501b65 Tests: Drop test for VS 7.0 generator deprecation warnings
| * | Deprecate Visual Studio 7 .NET 2003 generatorBrad King2016-04-282-1/+18
| |/ | | | | | | | | Update documentation to mark the generator deprecated. Add a warning at the end of generation plus an option to turn off the warning.
* | Merge topic 'clang-tidy-argv0'Brad King2016-04-291-2/+4
|\ \ | | | | | | | | | | | | 304ab71d Clang-Tidy: Give entire compiler command line to clant-tidy tool
| * | Clang-Tidy: Give entire compiler command line to clant-tidy toolDaniel Pfeifer2016-04-281-2/+4
| |/ | | | | | | | | The tool expects the entire compiler command, including `cc`. It will automatically skip that and extract the options it needs.
* | CMake Nightly Date StampKitware Robot2016-04-291-1/+1
|/
* Merge topic 'clang-tidy-output-cleanup'Brad King2016-04-281-3/+7
|\ | | | | | | | | 04d74a7f Clang-Tidy: copy stdout to sterr; ignore original stderr
| * Clang-Tidy: copy stdout to sterr; ignore original stderrDaniel Pfeifer2016-04-271-3/+7
| | | | | | | | | | | | | | | | | | | | | | Clang-Tidy writes the number of warnings, the number of suppressed warnings, and instructions on how to suppress warnings to stderr. Since each source file is checked individually, this repetitive information is disturbing and should be suppressed. The actual warning messages are written to stdout. Some IDEs (eg. QtCreator) analyze only stderr for issues. Redirecting Clang-Tidy's stdout to stderr makes sure the warnings are correctly displayed.
* | CMake Nightly Date StampKitware Robot2016-04-281-1/+1
| |
* | Merge topic 'ghs-duplicate-objects'Brad King2016-04-272-0/+44
|\ \ | | | | | | | | | | | | 1703a6d2 GHS: Fix handling of duplicate source filenames (#16046)
| * | GHS: Fix handling of duplicate source filenames (#16046)Geoff Viola2016-04-252-0/+44
| | | | | | | | | | | | | | | Green Hills MULTI project files must specify explicitly distinct object file names for source files with the same name.
* | | Merge topic 'autogen-message-cleanup'Brad King2016-04-273-113/+174
|\ \ \ | | | | | | | | | | | | | | | | | | | | 9647af3f Autogen: Message tweaks: Compiler type (moc/qrc/ui) added to progress messages 9b58190c Autogen: Message cleanups: Compose messages in std::stringstream
| * | | Autogen: Message tweaks: Compiler type (moc/qrc/ui) added to progress messagesSebastian Holtermann2016-04-251-4/+4
| | | |
| * | | Autogen: Message cleanups: Compose messages in std::stringstreamSebastian Holtermann2016-04-253-109/+170
| |/ / | | | | | | | | | | | | To avoid Race conditions with other processes writing to stdout/stderr compose the whole message in a std::stringstream then submit the single complete message.
* | | Merge topic 'vs-old-express-no-folders'Brad King2016-04-278-17/+28
|\ \ \ | | | | | | | | | | | | | | | | | | | | 571bedec VS: Ignore USE_FOLDER property on VS versions that do not support it 0c58d2d0 VS: Detect VS 8 and 9 Express editions
| * | | VS: Ignore USE_FOLDER property on VS versions that do not support itBrad King2016-04-267-13/+16
| | | | | | | | | | | | | | | | | | | | Solution folders are supported on VS 8 and above in the full versions and on VS 11 and above in the express versions.
| * | | VS: Detect VS 8 and 9 Express editionsBrad King2016-04-264-4/+12
| |/ /