[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