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

c4e8126a Drop unused CXX cstdio check
74daf33b Drop unused Borland and OpenWatcom preprocessor conditions
2b749749 Drop unused C type macros check
441dd494 Drop unused checks for 'long long' and '__int64'
a0bd892c SystemInformation: Use 'long long' unconditionally

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !192

The `KWSYS_CXX_HAS_CSTDIO` check has been unused since commit 363bf5ab
(Configure: Drop Large File Support macro definitions, 2019-02-22).

The versions of these compilers previously supported do not
support C++11 that we now require.

The `KWSYS_C_TYPE_MACROS` check has not been used since commit c1149ef6
(Drop the FundamentalType.h component of KWSys, 2015-12-14).

Now that KWSys requires C++11, we can assume `long long` and `std:atoll`
are available.

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

ce4941c9 IOStream: Drop this component of KWSys

Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: Ben Boeckel <ben.boeckel@kitware.com>
Merge-request: !190

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

49d04c2c hashtable: Drop this component of KWSys

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !191

It was already turned off when building inside CMake.
Other consuming projects have been migrated to C++11 and
can now use `std::unordered_{set,map}` instead.

The `IOStream` component was meant to provide compatibility with old C++
stream implementations that did not support `long long` and/or
`__int64`.  Now that KWSys requires C++11 that is not a problem.

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

25775078 Glob: Do not warn if top-level directory does not exist

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !189

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

e093a985 Glob: Append warning message only if destination is non-null

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !188

Fix the change from commit f33ef4e3 (Glob: issue warnings on Directory
errors, 2020-05-12) to not append to nullptr messages.

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

97f880e6 Directory: add extra tests involving errorMessage
f33ef4e3 Glob: issue warnings on Directory errors
9a3f9799 Directory: implement error reporting

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !187

The current implementation HandleGlobImpl with throw fatal errors for
all Glob::MessageType other than cyclicRecursion. This will need to be
addressed in cmake.

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

775296c8 Applied clang-tidy bugprone-suspicious-string-compare fixes
2c179921 Applied clang-tidy modernize-deprecated-headers
cb541c31 Auto-fixed various clang-tidy warnings

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !185

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

a19f0ac6 Fixed clang -Wstrict-prototypes warning

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !186

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

c58d4b47 SystemTools: On Windows, strip 'e' from Fopen mode

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !184

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

The MSVC runtime does not support the 'e' mode specifier.

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

00629420 SystemInformation: Add support for Windows on ARM64

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !183

Hernan Martinez authored 4 years ago and Brad King committed 4 years ago

Update preprocessor defines and intrinsics.

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

b8177b56 testProcess: Disable unreliable test case 7

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !182

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

39fd4817 testProcess: fix another clang-tidy warning

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !181

This test fails very frequently on CI because it relies on the
timing of process scheduling by the OS, which is not deterministic.
Disable it.

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

de210648 clang-tidy: address `readability-else-after-return` lint
535633fa clang-tidy: address `readability-isolate-declaration` lint
83b20b65 clang-tidy: address `readability-braces-around-statements` lint

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !180

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

986519af SystemTools: handle splitting a string starting with the separator
09942f51 testSystemTools: add tests for SplitString

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !178

Ben Boeckel authored 4 years ago and Brad King committed 4 years ago

If a string started with the separator, the first component would have
the separator included in it.