[reportlab-users] Fwd: ReportLab problems with CJK

[Robin Becker]

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

>