[Lldb-commits] [PATCH] D30054: Delete DataBufferMemoryMap

[Zachary Turner via lldb-commits]

I left out unit tests since we'd essentially be duplicating the unit tests
of MemoryBuffer, and because it involves the file system (also this is
temporary code until DataBuffer stuff goes away). Lmk if you disagree
though
On Fri, Feb 24, 2017 at 2:53 AM Pavel Labath via Phabricator <
reviews at reviews.llvm.org> wrote:

> labath added a comment.
>
> I am not sure if this is a voting situation, but I agree with what Zachary
> said above.
>
> Since we're already speaking about tests, it looks like the new
> DataBufferLLVM class could use a unit test or two, just so we get in the
> habit of writing those.
>
>
> https://reviews.llvm.org/D30054
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20170224/1e6f7891/attachment.html>