summaryrefslogtreecommitdiffstats
path: root/Source/CursesDialog
Commit message (Collapse)AuthorAgeFilesLines
* Remove unnecessary c_str() callsVitaly Stakhovsky2018-09-051-1/+1
| | | | Use the new IsOn(),IsOff() overloads.
* cmCursesLongMessageForm: Avoid -Wstringop-overflow warningBrad King2018-07-311-1/+1
| | | | | | | | We use `strncpy` to copy the title up to a maximum number of characters. GCC 8's `-Wstringop-overflow` warns that the length depends on the input length because it fails to recognize that we are bounding it to the buffer size too. Update the code to hide the dependence on the input length.
* Fix misc. typosluz.paz2018-06-042-2/+2
| | | | Found via `codespell` and `grep`
* Revise C++ coding style using clang-format-6.0Kitware Robot2018-06-013-21/+27
| | | | | | | | | | | | Run the `clang-format.bash` script to update all our C and C++ code to a new style defined by `.clang-format`. Use `clang-format` version 6.0. * 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.
* clang-format.bash: update to clang-format-6.0Brad King2018-06-011-1/+1
| | | | | | | | | | | | Update `.clang-format` with configuration to make the 6.0 format as close as possible to what 3.8 produced before. Then revise the style: * Indent preprocessor directives (a feature new since 3.8) * Add a newline and indentation before inheritance `:` and `,` Rename the Git attribute identifying the format to include the clang-format version number: `format.clang-format-6.0`. This will aid external infrastructure in knowing what version of the tool to run.
* ccmake: Fix compilation with ncurses on SolarisBrad King2018-03-201-0/+13
| | | | | | | | | | | | | | | | | | | On Solaris the ncurses header may define an `__attribute__` macro. This breaks C++ headers that use `__attribute__(...)` syntax. Somehow it causes references to unresolved symbols: __gthrw_pthread_once __gthrw_pthread_mutex_lock __gthrw_pthread_mutex_unlock instead of references to the correct symbols: pthread_once pthread_mutex_lock pthread_mutex_unlock Detect this case and undefine the `__attribute__` macro after including the curses headers.
* Merge topic 'ccmake-stack-smashing'Brad King2018-03-131-1/+1
|\ | | | | | | | | | | | | 7899e53691 ccmake: fix status line buffer overflow on very wide terminals Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1836
| * ccmake: fix status line buffer overflow on very wide terminalsTianhao Chai2018-03-131-1/+1
| | | | | | | | | | A mistyped length calculation will memset across stack frame when the user's terminal width is larger than 270.
* | Extend libuv file translate mode workaround to all executablesBrad King2018-01-231-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since libuv commit v1.14.1~7 (win: add uv__once_init() calls, 2017-08-30) the libuv initialization of the file translate mode may take place even if we do not use a uv loop. This change was included in our libuv update commit f4a26c748b (libuv 2018-01-19). Therefore use of libuv even through `cmSystemTools::GetRealPath` in any executable may trigger its file translate mode setting. Factor out the logic added to `cmake.exe` by commit v3.9.0-rc4~10^2 (cmake: Fix default file translate mode when using libuv, 2017-06-13) and re-use to initialize all executables. Issue: #16962
* | cmake: specify source file extensionsBen Boeckel2018-01-091-13/+13
|/
* CursesDialog: avoid calling strcpy()Rolf Eike Beer2017-09-251-33/+16
| | | | | Also use memset() and a few places where the compiler will collapse the for loop into such a call anyway.
* Enable clang-tidy modernize-loop-convert lintBrad King2017-09-193-9/+5
| | | | | Fix remaining diagnostics by this lint and remove it from our list of disabled lints.
* Use C++11 override instead of CM_OVERRIDEBrad King2017-09-158-20/+20
| | | | | | | | We now require C++11 support including `override`. Drop use of the old compatibility macro. Convert references as follows: git grep -l CM_OVERRIDE -- '*.h' '*.hxx' '*.cxx' | xargs sed -i 's/CM_OVERRIDE/override/g'
* Merge topic 'cstyle-casts'Daniel Pfeifer2017-08-301-1/+1
|\ | | | | | | | | | | | | 190e3825 Replace C-style casts Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1176
| * Replace C-style castsDaniel Pfeifer2017-08-271-1/+1
| |
* | IWYU: Mark cmConfigure.h with pragma: keepDaniel Pfeifer2017-08-2614-14/+13
|/ | | | Also remove `#include "cmConfigure.h"` from most source files.
* Use C++11 nullptrDaniel Pfeifer2017-08-248-44/+43
|
* CursesDialog: use target_include_directories for cmFormDaniel Pfeifer2017-08-182-11/+10
|
* Access string npos without instancePavel Solodovnikov2017-06-011-1/+1
|
* cmake: initialize with Role that controls which commands to registerDaniel Pfeifer2017-05-082-2/+2
|
* Use CM_DISABLE_COPYDaniel Pfeifer2017-04-2415-48/+28
|
* clang-tidy: remove else after break and continueDaniel Pfeifer2017-04-212-4/+4
|
* Use quotes for non-system includesDaniel Pfeifer2017-04-1118-20/+20
| | | | | | | | | | | | | Automate with: git grep -l '#include <cm_' -- Source \ | xargs sed -i 's/#include <\(cm_.*\)>/#include "\1"/g' git grep -l '#include <cmsys/' -- Source \ | xargs sed -i 's/#include <\(cmsys\/.*\)>/#include "\1"/g' git grep -l '#include <cm[A-Z]' -- Source \ | xargs sed -i 's/#include <\(cm[A-Z].*\)>/#include "\1"/g'
* clang-format.bash: Use Git attributes to mark files for formattingBrad King2017-02-221-0/+1
|
* clang-tidy: apply modernize-use-bool-literals fixesDaniel Pfeifer2016-12-123-10/+6
|
* ccmake: Correctly display edit mode and associated keysSylvain Joubert2016-11-152-49/+46
| | | | | | | | Specific widgets can implement `PrintKeys` but their effects were hidden by the main widget `PrintKeys`. Fix this to give the user a better hint about the edit mode status. Closes: #13757
* ccmake: Add missing spaces to match the previous text lengthSylvain Joubert2016-11-151-1/+1
| | | | There was a "glitch" leaving "it" on the screen
* cmListFileCache: Remove cmState header includeStephen Kelly2016-10-192-0/+2
| | | | | Include it in dependents which have previously relied on it transitively.
* cmState: Port dependents to new cmStateTypes headerStephen Kelly2016-10-1910-10/+10
|
* cmState: Move CacheEntryType enum to separate namespaceStephen Kelly2016-10-1910-30/+31
| | | | Port dependent code to the change.
* Simplify CMake per-source license noticesBrad King2016-09-2729-320/+59
| | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
* ccmake: Fix recent compilation regression with GCC on SolarisBrad King2016-09-142-8/+0
| | | | | | | The change in commit 32f756c8 (CursesDialog: include what you use, 2016-09-01) revealed that an ancient workaround for compiling with GCC on Solaris has not had an effect in a long time and is now incorrect. Drop it.
* CursesDialog: include what you useDaniel Pfeifer2016-09-0326-36/+116
|
* make sure to include cmConfigure.h before cmStandardIncludes.hDaniel Pfeifer2016-09-031-0/+2
|
* CursesDialog: don't use else after returnDaniel Pfeifer2016-08-184-32/+27
|
* Merge topic 'readability-named-parameter'Brad King2016-08-178-13/+20
|\ | | | | | | | | e7b842e1 Make sure unnused parameters are /*named*/
| * Make sure unnused parameters are /*named*/Daniel Pfeifer2016-08-168-13/+20
| |
* | fix a batch of include-what-you-use violationsDaniel Pfeifer2016-08-161-0/+2
|/
* ccmake: Add VIM-like bindings for navigationPaul Seyfert2016-08-023-12/+23
| | | | | | * scroll with j/k * toggle bool with space, enter insert mode with i * bindings not shown at the bottom of the screen, but given in help
* ccmake: Revise documentation for [d]Paul Seyfert2016-08-021-2/+2
| | | | | * list it at the bottom of the screen * different place in help message
* ccmake: Fix typo in help (it's -> its)Paul Seyfert2016-08-021-1/+1
|
* use CM_NULLPTRDaniel Pfeifer2016-06-288-42/+43
|
* mark functions with CM_OVERRIDEDaniel Pfeifer2016-06-278-20/+20
|
* Add missing braces around statements.Daniel Pfeifer2016-06-101-2/+2
| | | | | Apply fixits of clang-tidy's readability-braces-around-statements checker.
* Merge topic 'simplify-boolean-expressions'Brad King2016-06-031-5/+1
|\ | | | | | | | | 7f6b8d33 Simplify boolean expressions
| * Simplify boolean expressionsDaniel Pfeifer2016-06-021-5/+1
| | | | | | | | | | | | Use clang-tidy's readability-simplify-boolean-expr checker. After applying the fix-its, revise all changes *very* carefully. Be aware of false positives and invalid changes.
* | Add option to control 'bin' directory of CMake's own installation (#16076)Nicolas BUNEL2016-06-011-1/+1
|/ | | | | Add a `CMAKE_BIN_DIR` cache entry to CMake's own build configuration. Add a `--bindir` option to the `bootstrap` script to set it.
* Improve string find: prefer character overloads.Daniel Pfeifer2016-05-241-1/+1
| | | | | Apply fix-its from clang-tidy's performance-faster-string-find checker. Ignore findings in kwsys.
* Revise C++ coding style using clang-formatKitware Robot2016-05-1623-1153/+803
| | | | | | | | | | | | | 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-4/+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.