Re: [Lldb-commits] [PATCH] D22755: Fix DataExtractor::PeekData for zero length peeks

2016-07-26 Thread Pavel Labath via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL276734: Fix DataExtractor::PeekData for zero length peeks (authored by labath). Changed prior to commit: https://reviews.llvm.org/D22755?vs=65339&id=65478#toc Repository: rL LLVM https://reviews.llv

Re: [Lldb-commits] [PATCH] D22755: Fix DataExtractor::PeekData for zero length peeks

2016-07-25 Thread Greg Clayton via lldb-commits
clayborg accepted this revision. clayborg added a comment. This revision is now accepted and ready to land. Looks good. https://reviews.llvm.org/D22755 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/l

[Lldb-commits] [PATCH] D22755: Fix DataExtractor::PeekData for zero length peeks

2016-07-25 Thread Pavel Labath via lldb-commits
labath created this revision. labath added a reviewer: clayborg. labath added a subscriber: lldb-commits. The function was returning the null pointer for peeks of size zero, which seems like a sensible thing to do, but is actually pretty easy to get bitten by that if you are extracting a variabl