Re: Patch to support tables with oversize cells

Lennart Regebro via reportlab-users <[email protected]> Mon, 30 May 2022 15:24:41 +0200
Newsgroups gmane.comp.python.reportlab.user
Message-ID <CAHT-kBcdiwWmhF_i-wAS9AQVbAU7mksyaxGF+K6Kg9Z2XBgniQ@mail.gmail.com>
On Mon, May 30, 2022 at 3:01 PM Robin Becker <[email protected]> wrote:

> For whatever reason we have T is self in the non-splitInRow case so we
> have to be careful about assignments to T.attr in
> the code executed by that branch.
>

Ah, yes, of course, the penny dropped now. I forgot about that, and
probably ended up setting things on T without thinking about it.


> I think on the other branch we create a new T, but it's not clear that we
> are setting stuff back on self unless some
> mutable is shared and then modified.
>

I'll make a code review in this with this insight, and come back to you,
likely with a verion 8.