Alright, already found a regression with ToolBuilder-Morphic-ct.279. In <font color="#000080">#</font><font color="#000000">(((</font><font color="#000000">))</font><font color="#000000"> </font><font color="#000000">((</font><font color="#000000">)))</font>, you cannot select the last leaf any longer.<br>
<br>
Hm, nevertheless this is a sad restriction. In pluggable lists (by item), we also use value equality/#= (see PluggableListMorphByItem >> #getCurrentSelectionIndex); this is just not consistent. Maybe ObjectExplorer et al. should wrap every line into a unqie object instead?<br>
<br>
Performance could also be an issue, but I would rather leave this decision up to the specific domain ...<br>
<br>
Best,<br>
Christoph<br>
<br>
<font color="#808080">---<br>
</font><i><font color="#808080">Sent from </font></i><i><u><a href="https://github.com/hpi-swa-lab/squeak-inbox-talk"><font color="#808080">Squeak Inbox Talk</font></a></u></i><br>
<br>
On 2021-10-26T11:03:34-07:00, tim@rowledge.org wrote:<br>
<br>
> The issue I can imagine here is the potentially high cost of #= compares. After all, comparing a large directory to another might easily be a deeply recursive problem with added fun of links to other filing systems on remote machines with damp-string grade network connections. Not that I've ever made that mistake. No sir, never, not me.<br>
> <br>
> > On 2021-10-25, at 9:59 PM, christoph.thiede at student.hpi.uni-potsdam.de wrote:<br>
> > <br>
> > Hi all!<br>
> > <br>
> > I am just fixing a bug in the PluggableTreeMorph and was stumbling upon the fact that in #updateFromChangedObject:, items are compared by identity, which makes it harder in the context of a DirectoryChooserDialog to indicate the relevant instance (i.e., you cannot use #containingDirectory which would create a new copy).<br>
> > <br>
> > Do we actually need identity (#==) instead of equality here? Next to #updateFromChangedObject:, also #getCurrentSelectionItem is using #==. As opposed to this, #selectPath:in: uses #=. Unless you have any counter arguments, I would vote for #= in all three places to delegate equality control to the domain objects that are contained in the tree morph.<br>
> > <br>
> > Best,<br>
> > Christoph<br>
> > <br>
> > <br>
> > ---<br>
> > Sent from Squeak Inbox Talk<br>
> <br>
> <br>
> tim<br>
> --<br>
> tim Rowledge; tim at rowledge.org; http://www.rowledge.org/tim<br>
> A bug in the code is worth two in the documentation.<br>
> <br>
>