[Lldb-commits] [PATCH] D68647: Simplify LZMA decoding by using ArrayRef::take_back

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Oct 8 07:57:50 PDT 2019


labath added inline comments.


================
Comment at: lldb/source/Host/common/LZMA.cpp:99
+      &xzindex, &memlimit, nullptr,
+      InputBuffer.take_back(LZMA_STREAM_HEADER_SIZE - opts.backward_size)
+          .data(),
----------------
I think this should be + to preserve semantics, though I am not sure why no tests fail because of that. I guess none of the lzma streams we have have non-zero `backward_size`..


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D68647/new/

https://reviews.llvm.org/D68647





More information about the lldb-commits mailing list