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.

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

Hans Johnson authored 7 years ago and Brad King committed 7 years ago

When building with C++11 or greater, use the `nullptr` keyword.

Fix a return value accidentally changed by commit cff58f07
(RegularExpression: New RegularExpressionMatch class, 2017-12-04).

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.

RegularExpression::find() used to use shared global
variables for every instance. This moves the variables
into a private class on the stack instead.

RegularExpression::compile() used to use shared global
variables for every instance. This moves the variables
into a private class on the stack instead.

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

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

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

Also adjust nearby whitespace.

Change-Id: I83a17103c3e5c896c30226fc4d9d8b5b94c66e59

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.

STYLE:Empty commit just add information about rev 1.15: the reason is that long is 64-bit on gcc on Linux because it uses the LP64 data model whereas long is 32-bit on VS 64-bit because it uses the LLP64 model (ref: http://en.wikipedia.org/wiki/64-bit#64-bit_data_models )

COMP:Fix warning on VS 64bit. Don't why gcc 4.3.2 didn't catch this one on a 64bit machine with -Wconversion on.

COMP: Adding work-around for CMake dependency scanning limitation.  Any configured header included by KWSYS_HEADER() in a .c or .cxx file in kwsys itself must use this hack to get dependencies.

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

ENH: Added use of KWSYS_HEADER macro in c and cxx files to include kwsys headers through their configured namespace.