[PATCH] D105292: [NFC] Mark Expected<T>::assertIsChecked() as const
Sam McCall via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 6 23:34:25 PDT 2021
sammccall accepted this revision.
sammccall added a comment.
This revision is now accepted and ready to land.
LG thanks!
I guess this didn't cause problems so far because usage of const Expected is rare.
(next time please upload with full context)
================
Comment at: llvm/unittests/Support/ErrorTest.cpp:574
+ "Expected<T> must be checked before access or destruction.")
+ << "Unchecekd Expected<T> success value did not cause an abort().";
}
----------------
Nit: unchecked (typo)
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105292/new/
https://reviews.llvm.org/D105292
More information about the llvm-commits
mailing list