================ @@ -777,7 +779,7 @@ template <> struct MappingTraits<FormatStyle::SpacesInLineComment> { IO.mapOptional("Maximum", signedMaximum); Space.Maximum = static_cast<unsigned>(signedMaximum); - if (Space.Maximum != -1u) + if (Space.Maximum != std::numeric_limits<unsigned>::max()) ---------------- alsepkow wrote:
On further consideration, I think it would be cleaner to stay consistent with using numeric_limits for all of the updates. And given that there are only a handful of UINT_MAX cases in this file I've swapped them to use numeric_limits for consistency. That would be the preferred choice if this were new code anyways. https://github.com/llvm/llvm-project/pull/147623 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits