<div dir="ltr"><div>Hello Jim,</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"> <span style="font-size:12.8000001907349px">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.</span></blockquote><div>I assumed that ValueDidChange is tracking value since it was created.It isn't obvious for me when I can use GetValueDidChange. I think this comment will help other people that think like me.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span style="font-size:12.8000001907349px">That might be a better comment, since it says what actually happens, if you think a comment is needed here.</span> </blockquote><div>If you think that it's obvious I can remove it. Now I'm familiar with that and I don't need them.<br></div><div><br></div><div>Thanks,</div><div>Ilia</div><div> </div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Mar 7, 2015 at 1:53 AM, Jim Ingham <span dir="ltr"><<a href="mailto:jingham@apple.com" target="_blank">jingham@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">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.<br>
<br>
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.<br>
<br>
That might be a better comment, since it says what actually happens, if you think a comment is needed here.<br>
<span class=""><br>
<br>
REPOSITORY<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D8103" target="_blank">http://reviews.llvm.org/D8103</a><br>
<br>
</span>EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</blockquote></div><br></div>