================
@@ -528,7 +528,11 @@ SuspiciousCallArgumentCheck::SuspiciousCallArgumentCheck(
for (const StringRef Abbreviation : optutils::parseStringList(
Options.get("Abbreviations", DefaultAbbreviations))) {
const auto [Key, Value] = Abbreviation.split("=");
- assert(!Key.empty() && !Value.empty());
----------------
zeyi2 wrote:
IMO we shouldn't assert here. IIUC Assertions are typically for invariant
checking and logic errors, not for validating user input.
https://github.com/llvm/llvm-project/pull/180351
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
