Hi Eliot,

Seeing that the commit is called "fix debugging of optimized expressions", I remembered  that I had recently stumbled
upon a debugger annoyance related to an optimized expression.
I checked and unfortunately it is still there, so if you fell inspired :)

One easy way to see it is to debug CompiledMethod>>#= , comparing a compiled method to itself.
In the to:do: loop, at every iteration, the debugger highlights the early exit "^false", even though it does not
actually execute it.

Cheers,
Florin



 
On 10/6/2014 7:47 PM, Eliot Miranda wrote:
>
>
> On Mon, Oct 6, 2014 at 3:58 PM, David T. Lewis <lewis at mail.msen.com <mailto:lewis at mail.msen.com>> wrote:
>
>     On Mon, Oct 06, 2014 at 03:44:52PM -0700, Eliot Miranda wrote:
>     > On Mon, Oct 6, 2014 at 3:43 PM, Eliot Miranda <eliot.miranda at gmail.com <mailto:eliot.miranda at gmail.com>> wrote:
>     > >
>     > > On Mon, Oct 6, 2014 at 2:43 PM, Bal??zs K??si <kosi.balazs at ezomer.hu <mailto:kosi.balazs at ezomer.hu>> wrote:
>     > >
>     > >> Thanks Eliot for fixing this!
>     > >>
>     > >> I've updated my image and now anything I try to debug results in the
>     > >> popup: 'Nothing to debug; expression is optimized', so I removed the code
>     > >> creating the popup, and can confirm that the first issue is gone.
>     > >
>     > > Doh!  I forgot to test that debug it still works, and of course broke it.
>     > > OK, there's a new fix in place now.  I do apologize.
>     >
>     > Ugh, I'm having a bad day.  I just committed something that exhibited the
>     > old symptom.  Sorry.  Give me a few to get this right.
>
>     Eliot,
>
>     You on a bad day would be way better than me on a good day, and I'm probably
>     not the only one who would say that. Thanks very much for addressing these
>     issues, and thanks for keeping the MVC side updated too :-)
>
>
> thanks for the kind words.  I wish I was that good ;-).  But fingers crossed there's a proper fix there now.  If it's
> easy please test the MVC changes; TIA.
> -- 
> best,
> Eliot
>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20141006/1dbc486f/attachment.htm