seanm wrote:
I tried with [ITK](https://github.com/InsightSoftwareConsortium/ITK/) and the
resulting transformation did not compile.
Many changes were of this form:
```diff
- const double Max = 1.0 - Min;
+ constexpr double Max = 1.0 - Min;
```
Which is great, though notice the double space after `constexpr` for some
reason.
Other changes were like this:
```diff
- const auto check = [](const auto & ptr) { EXPECT_THROW(itk::Deref(ptr),
itk::DerefError); };
+ constexpr const auto check = [](auto & ptr) { EXPECT_THROW(itk::Deref(ptr),
itk::DerefError); };
```
I'm no C++ expert, but is it right to have both `const` ***and*** `constexpr`
here?
Also, I was surprised to see the 2nd const *removed*. And this removal
generates one of the many compiler errors.
Still, this is looking like it'll be great!
https://github.com/llvm/llvm-project/pull/146553
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
