Re: [Lldb-commits] [lldb] r255237 - Differential Revision: http://reviews.llvm.org/D15333

2015-12-10 Thread Aidan Dodds via lldb-commits
Hi Tamas, Your right, sorry about that, I will be sure to add a commit message in the future. Thanks, Aidan On 10/12/2015 11:10, Tamas Berghammer wrote: Hi Aidan, Next time please add a commit message describing the change itself (next to the review link). In general I think copying the re

Re: [Lldb-commits] [lldb] r255237 - Differential Revision: http://reviews.llvm.org/D15333

2015-12-10 Thread Tamas Berghammer via lldb-commits
Hi Aidan, Next time please add a commit message describing the change itself (next to the review link). In general I think copying the review title and the summary into the commit message is a good baseline. It will help us understand your change when somebody looking at git logs as we don't have

[Lldb-commits] [lldb] r255237 - Differential Revision: http://reviews.llvm.org/D15333

2015-12-10 Thread Aidan Dodds via lldb-commits
Author: aidandodds Date: Thu Dec 10 04:11:49 2015 New Revision: 255237 URL: http://llvm.org/viewvc/llvm-project?rev=255237&view=rev Log: Differential Revision: http://reviews.llvm.org/D15333 Modified: lldb/trunk/source/Core/DataExtractor.cpp Modified: lldb/trunk/source/Core/DataExtractor.cpp