[squeak-dev] Change in LazyListMorph breaks
MulticolumnLazyListMorph.
Chris Muller
asqueaker at gmail.com
Thu Dec 15 16:23:16 UTC 2011
Perhaps neither. One thing that would help is to tell us a way to
reproduce the issue you're seeing -- as I cannot see any way
McLLM>>drawOn: could die with non-indexable -- perhaps you meant
setColumnWidthsFor: died?
In any case, MulticolumnLazyListMorph is not being used inside the
image. So lets look to fixing it rather than rolling back to
requiring all Strings in all lists.
- Chris
On Tue, Dec 13, 2011 at 4:35 PM, Christopher Oliver
<current.input.port at gmail.com> wrote:
> Change Morphic-cmm.523 as merged in Morphic-ul.524 breaks MulticolumnLazyListMorph. The
> rows for a McLLM are arrays of strings, and because LazyListMorph, as it is now, converts
> these to strings, McLLM>>drawOn: dies with non-indexable. I.e. first index gets the
> character $# (the start of an array literal), and then deeper, we try to index it again.
> Ugh! Should McLLM just do its own thing, and not subclass LLM, or should we roll back
> Muller's change on LLM?
>
> Thoughts?
>
> --
> Christopher Oliver <current.input.port at gmail.com>
>
More information about the Squeak-dev
mailing list
|