[PATCH] D63713: Add error handling to the DataExtractor class
Pavel Labath via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 25 05:19:17 PDT 2019
labath requested review of this revision.
labath added a comment.
I finally got around to updating the patch for size_t. It ended up looking slightly uglier than I hoped for (like the need for a special `isValidOffsetForDataOfSizeT`), but OTOH I also was able to remove the default-null error arguments from the "legacy" APIs.
Overall I don't think this can be made much cleaner without porting existing users to size_t first. Please LMK what you think.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D63713/new/
https://reviews.llvm.org/D63713
More information about the llvm-commits
mailing list