================ @@ -1877,8 +1880,9 @@ struct AdditionalKeywords { // In Verilog the colon in a default label is optional. return Tok.is(TT_CaseLabelColon) || (Tok.is(tok::kw_default) && - !(Next && Next->isOneOf(tok::colon, tok::semi, kw_clocking, kw_iff, - kw_input, kw_output, kw_sequence))); + (!Next || + Next->isNotOneOf(tok::colon, tok::semi, kw_clocking, kw_iff, + kw_input, kw_output, kw_sequence))); ---------------- owenca wrote:
For pointers, I prefer `!(Next && Next->isOneOf(A, B)` to `!Next || Next->isNotOneOf(A, B)` because the `Next &&` part is just for avoiding dereferencing a null pointer. https://github.com/llvm/llvm-project/pull/161021 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits