Re: [Lldb-commits] [PATCH] D24126: Make Scalar::GetValue more consistent

2016-09-02 Thread Pavel Labath via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL280476: Make Scalar::GetValue more consistent (authored by labath). Changed prior to commit: https://reviews.llvm.org/D24126?vs=69979&id=70140#toc Repository: rL LLVM https://reviews.llvm.org/D24126

Re: [Lldb-commits] [PATCH] D24126: Make Scalar::GetValue more consistent

2016-09-01 Thread Zachary Turner via lldb-commits
Actually you're right, i read the test backwards and thought the string was the argument. I shouldn't do this 30 seconds after waking up :) On Thu, Sep 1, 2016 at 7:26 AM Pavel Labath wrote: > I don't think that even makes sense. :) > > Scalar class has constructors overrides for each of the prim

Re: [Lldb-commits] [PATCH] D24126: Make Scalar::GetValue more consistent

2016-09-01 Thread Pavel Labath via lldb-commits
I don't think that even makes sense. :) Scalar class has constructors overrides for each of the primitive types. I cannot construct it with a value which is larger than it will fit in the primitive type. I suppose I could try the boundary conditions like UINT32_MAX, but that's about it. Or I am m

Re: [Lldb-commits] [PATCH] D24126: Make Scalar::GetValue more consistent

2016-09-01 Thread Zachary Turner via lldb-commits
What about a case in the unit test where the value overflows the type? Would that be useful? On Thu, Sep 1, 2016 at 3:50 AM Pavel Labath via lldb-commits < lldb-commits@lists.llvm.org> wrote: > labath created this revision. > labath added reviewers: clayborg, granata.enrico. > labath added a subsc

[Lldb-commits] [PATCH] D24126: Make Scalar::GetValue more consistent

2016-09-01 Thread Pavel Labath via lldb-commits
labath created this revision. labath added reviewers: clayborg, granata.enrico. labath added a subscriber: lldb-commits. It seems the original intention of the function was printing signed values in decimal format, and unsigned values in hex (without the leading "0x"). However, signed and unsigne