<div dir="auto">Thanks, Marcel!</div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed 8. Dec 2021 at 18:05 <<a href="mailto:marcel.taeumel@hpi.de">marcel.taeumel@hpi.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)">Revised and merged. See Tools-mt.1084.<br>
<br>
On 2021-05-17T10:52:10+02:00, <a href="mailto:lists@fniephaus.com" target="_blank">lists@fniephaus.com</a> wrote:<br>
<br>
> Thanks, Christoph! The changeset looks good to me and fixes our issue.<br>
> Nonetheless, it's probably better if Marcel reviews and merges it.<br>
> Marcel, could you have a look please?<br>
> <br>
> Many thanks,<br>
> Fabio<br>
> <br>
> On Fri, May 14, 2021 at 4:29 PM Thiede, Christoph<br>
> <Christoph.Thiede at <a href="http://student.hpi.uni-potsdam.de" rel="noreferrer" target="_blank">student.hpi.uni-potsdam.de</a>> wrote:<br>
> ><br>
> > Hi all!<br>
> ><br>
> ><br>
> > Jan Ehmueller and Fabio (fn) have discovered an Inspector bug with #inspectOne/"inspect element..." in the field list menu that hindered users from entering the index of a truncated element in the inspect element dialog. Possible ways to reproduce the issue include:<br>
> ><br>
> ><br>
> > Inspector openOn: (Array new: 1024) -> … -> Click -> 90 -> Enter<br>
> > (SmalltalkImage>>#snapshot:andQuit:withExitCode:embedded:) inspect -> right click self -> inspect element... -> 450 -> Enter (might depend on Sista)<br>
> > TruffleSqueak was also affected: <a href="https://github.com/hpi-swa/trufflesqueak/issues/143" rel="noreferrer" target="_blank">https://github.com/hpi-swa/trufflesqueak/issues/143</a><br>
> ><br>
> ><br>
> > The attached changeset attempts to fix the bug.<br>
> ><br>
> > Changelog:<br>
> ><br>
> > Problem: The variant of Inspector >> #inspectOne, as it is now in the trunk, fails if the key is the value for a field that has been truncated. Because in this place, explicitly only the keys of self fields are passed to #inspectOneOf: which are already truncated. The bug does not affect CollectionInspectors which passes the elementIndices directly to #inspectOneOf: without the need of having fields.<br>
> > Solution: I have now essentially pushed this logic from CollectionInspector with #elementIndices up to Inspector. This should make it run again.<br>
> ><br>
> > Please review and merge if you don't have any further objections. :-)<br>
> ><br>
> > Best,<br>
> > Christoph<br>
> ><br>
> ><br>
> <br>
> <br>
<br>
</blockquote></div></div>