The compiler warns on use of `{}` initialization of array members:

    warning C4351: new behavior: elements of array '...' will be
                   default initialized

The `RegularExpressionMatch` class added by commit cff58f07
(RegularExpression: New RegularExpressionMatch class, 2017-12-04) has
members with array-of-pointer types.  Use uniform initialization
syntax to initialize all array elements on construction.

This fixes a Clang scan-build 7 warning:

```
RegularExpression.hxx:138:23: warning: The left operand of '==' is a garbage value
  if (this->startp[n] == nullptr) {
      ~~~~~~~~~~~~~~~ ^
```

Sean McBride authored 5 years ago and Brad King committed 5 years ago

- Find/replace of KWSYS_NULLPTR with nullptr
- Find/replace of NULL with nullptr
- Manually fixed -Wzero-as-null-pointer-constant warnings
- Careful review to fix some comments, string output, etc.

Kitware Robot authored 6 years ago and Brad King committed 6 years ago

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.

Found via `codespell`

+ fix documentation of method name
+ fix some source comment typos

The new RegularExpressionMatch allows to keep multiple match
results of a RegularExpression.

The startp and endp pointers of RegularExpression are replaced
by an instance of RegularExpressionMatch.

There also is new thread safe version of RegularExpression::find()
which allows multiple threads to concurrently use the same
RegularExpression on different input strings with independent
RegularExpressionMatch results.

luzpaz authored 7 years ago and Brad King committed 7 years ago

Found using

    codespell -q 3 --skip="./Utilities" -I .cmake-whitelist.txt`

in CMake whereby the whitelist contained:

    ans
    dum
    helpfull
    emmited
    emmitted
    buil
    iff
    isnt
    nto
    ot
    pathes
    substract
    te
    todays
    upto
    whitespaces

Kitware Robot authored 8 years ago and Brad King committed 8 years ago

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 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.

Change-Id: Ie3aedc03bfe5c3999d4a077fafb8c6f3bbffc5c5

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 KWSys 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#kwsys".

Run the `filter-notices.bash` script to perform the replacements mechanically.
Manually fix up the shebang line in one file.

Change-Id: I8497f7c868664dcf54a8608ab302ad93c860b334

_WIN32 is the actual define for Windows. WIN32 is a CMake-ism.

Change-Id: Iea9f69ab1719325703a3c9a6cacdd6ecf13dbc34

Drop "kwsys/stl/*" and "kwsys_stl::" and use the standard headers and
namespace instead.  Drop the now-unused KWSYS_STL_HAVE_STD check.

Change-Id: I6105fd32cd8f0e487e03f58927976e824a53f0d8

It's weird that find had one (non-inline) overload. Make overloads for
the ctor and .compile() as well so that other code can drop unnecessary
.c_str() calls.

Change-Id: If0da9811728892bdfd1340f51ec9b33524776958

This converts the KWSys 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 KWSys copyright to
cover the full development time range.

KWSys tries not to force anything on source files that include its
headers, but Borland warning 8027 leaves us no choice when we want to
have inline function definitions.  This commit disables the warning for
the RegularExpression header and any file that includes it.

BUG: if there is no match, don't construct the stl string from a NULL
pointer

Alex

ENH: Split kwsys_std into kwsys_ios and kwsys_stl in order to avoid std namespace pollution and support more platforms.