[Lldb-commits] [PATCH] Improve ValueObject::GetValueDidChange test; Add a comment for it

[Jim Ingham]

I don't see how the "It will only be valid the second time" comments help anything.  ValueDidChange can only mean "is it different from the last time you looked at it" since we aren't actually tracking all variable values in the program...  So by definition, the return from ValueDidChange is arbitrary the first time you look at an object.

Since we have to choose something, we set it to "false" when you first look at a value.  That is actually the correct choice because then a UI that is using ValueDidChange to drive the choice to color variables that have changed since the last stop in this frame won't set them all to the changed color the first time we hit the frame - which wouldn't be very useful.

That might be a better comment, since it says what actually happens, if you think a comment is needed here.


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D8103

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/