[reportlab-users] Patch to support tables with oversize cells

Lennart Regebro lregebro at shoobx.com
Sat Jun 4 06:19:33 EDT 2022


Great, thanks for the update!

On Sat, Jun 4, 2022 at 11:20 AM Robin Becker <robin at reportlab.com> wrote:

> On 01/06/2022 12:33, Lennart Regebro wrote:
> > Yes, two small bugs in height alignments that didn't show up in any
> > previous test. Good find.
> >
> >
> ..........
>
> Hi Lennart,
>
> I took a look at v9 and it seems fine. I'll try a few more tests and
> probably commit Monday.
>
> I made some minor changes to the patch.
>
> 0) I inlined the two matrixcopy calls
>
> 1) I moved the _cellvalues copy into the two different paths so the
> doInRowSplit test looks like
>
> if not doInRowSplit:
>      T = self
>      data = self._cellvalues
> else:
>      data = [_[:] for _ in self._cellvalues]
>
> I don't think the copy is required in the not doInRowSplit case; that
> lines up with the original code and my lines
> changed test code agrees.
> --
> Robin Becker
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://pairlist2.pair.net/pipermail/reportlab-users/attachments/20220604/3de3222f/attachment.htm>


More information about the reportlab-users mailing list