[reportlab-users] Fwd: ReportLab problems with CJK

Robin Becker robin at reportlab.com
Wed Feb 16 10:04:00 EST 2011


Hi Alexandr,

if you want to submit bug reports queries etc etc you can use the reportlab
users mailing list with a signup here

http://two.pairlist.net/mailman/listinfo/reportlab-users


Your mail is saying we should add a decrement to i after line 764; do you have a
fairly minimal example paragraph that illustrates the problem? I don't
speak/read Japanese, but Andy does and I'm sure a simple example will help
convince him.
--
Robin Becker


On 16/02/2011 11:21, Alexandr Vasilenko wrote:

> Hi Robin,

>

> I tried to send my message to support email, but got auto-respond that I'm

> not a customer. Though I found your email in mail lists, and your response

> on cjk splitting. I have some problems with it and propose fix

>

> Regards

> Alex

>

>

> ---------- Forwarded message ----------

> From: Alexandr Vasilenko<alexs.vasilenko at gmail.com>

> Date: 2011/2/16

> Subject: ReportLab problems with CJK

> To: support at reportlab.com, enquiries at reportlab.com

>

>

> Hello,

>

> My name is Alex Vasilenko, I'm currently working with ReportLab and have a

> chance to work on CJK report generation.

> Not sure, I'm writing to correct address, though as site says, support mail

> have filter for licensed customers only.

>

> Actually, I need my email to be redirected to developers. As i think there

> is a bug in next file:

> src/reportlab/platypus/paragraph.py (got from latest 10.10.2010 release)

> line 764 in *cjkFragSplit* method

>

> if endLine:

> extraSpace = maxWidth - widthUsed

> if not lineBreak:

> extraSpace += w

> #This is the most important of the Japanese typography

> rules.

> #if next character cannot start a line, wrap it up to this

> line so it hangs

> #in the right margin. We won't do two or more though -

> that's unlikely and

> #would result in growing ugliness.

> if i<nU:

> nextChar = U[i]

> if nextChar in ALL_CANNOT_START:

> extraSpace -= nextChar.width

> i += 1

>

> i think `i` variable decrement should be added:

>

> if endLine:

> extraSpace = maxWidth - widthUsed

> if not lineBreak:

> extraSpace += w

> *i -= 1*

> #This is the most important of the Japanese typography

> rules.

> #if next character cannot start a line, wrap it up to this

> line so it hangs

> #in the right margin. We won't do two or more though -

> that's unlikely and

> #would result in growing ugliness.

> if i<nU:

> nextChar = U[i*+1*]

> if nextChar in ALL_CANNOT_START:

> extraSpace -= nextChar.width

> i += 1

>

> I got the problem that text was overlapped over the table borders, and

> during reportlab debug I found such fix.

> So want to know, if it will be correct?

>

> Regards,

> Alex

>




More information about the reportlab-users mailing list