[squeak-dev] The Trunk: Tools-cmm.823.mcz
David T. Lewis
lewis at mail.msen.com
Tue Jun 26 02:20:05 UTC 2018
On Mon, Jun 25, 2018 at 07:04:12PM -0500, Chris Muller wrote:
> On Mon, Jun 25, 2018 at 6:40 PM, David T. Lewis <lewis at mail.msen.com> wrote:
> > This is probably just a mistake, or something missing from the commit.
> > But until that can be sorted out, the Tools-cmm.823 update should
> > probably be reverted and/or moved to the inbox.
>
> Nope, it's fine. There's not much code there, maybe take a look
> before making such a suggestion? You didn't say anything when I
> pointed out the original hack doesn't work (and still doesn't), but
> for whatever reason, you want to revert my code, which does work. :(
I have not followed this dicussion closely, so I checked back to try
to make sense of it. I am still confused.
Eliot made a change (Tools-eem.817) to have debugger windows open with
a window size matching that of previously opened debugger windows. This
may or amy not be a hack but it seems to work as advertised.
Chris pointed out a problem, apparently caused by Tools-eem.817, that
affects his image:
http://lists.squeakfoundation.org/pipermail/squeak-dev/2018-June/199157.html
I cannot say the cause of that problem, and I do not know how to reproduce
it, but it must be a problem otherwise Chris would not have raised the
concern.
Chris then introduced rememberExtent: in Tools-cmm.823, which apparently
is intended to provide a way to override Eliot's changes. But there
are no actual senders of rememberExtent: in the image.
Here is what I would say with confidence: A hack to work around
somebody else's hack does not belong in the trunk.
So I stand by my original statement that it would be good to revert
Tools-cmm.823 until the confusion can be sorted out.
Having said that, how can I actually reproduce the problem that Chris
reported earler in this post?
http://lists.squeakfoundation.org/pipermail/squeak-dev/2018-June/199157.html
Dave
>
> > As a general observation, it is a good idea for all of us to make
> > use of the inbox for changes that may potentially conflict with
> > someone else's work in progress. Merging from inbox to trunk is
> > easy, so it is never a bad idea to use the inbox when we are looking
> > for feedback or review on our changes.
>
> +1000
>
More information about the Squeak-dev
mailing list
|