[mgp-users-jp 01401] Re: wrong rendering with libm17n (Forward: Bug#302348: acknowledged by developer (Bug#302348: fixed in mgp 1.11b-2))

[Fumitoshi UKAI <ukai@debian.or.jp>]

At Fri, 13 May 2005 10:41:11 +0900 (JST),
Kenichi Handa wrote:
> > Date: Tue, 26 Apr 2005 23:44:26 -0700
> > From: Jeffery von Ronne <jronne@ics.uci.edu>
> > To: 302348@bugs.debian.org
> [...]
> > On Tue, Apr 19, 2005 at 10:03:19AM -0700, Debian Bug Tracking System wrote:
> >>  We believe that the bug you reported is fixed in the latest version of
> >>  mgp, which is due to be installed in the Debian FTP archive:
> [...]
> >>  Changes: 
> >>   mgp (1.11b-2) unstable; urgency=low
> >>   .
> >>     * build with --with-m17n-lib
> >>       add build-depends: libm17n-dev
> >>       closes: Bug#302348
> 
> > This does add working UTF-8 support but it also results in several
> > serious regressions:
> >   1. the xfont directives are being silently ignored and some
> >      other font is being substituted instead
> >   2. the quality of the font rendering is much worse
> >   3. wrapping of multi-line paragraphs is broken
> >      (a) centered paragraphs get left justified
> >      (b) paragraphs spliced together with the %cont directive don't
> >          wrap all the way to the left margin, but only to the location
> > 	 of the %cont directive
> 
> > I'm attaching a sample test.mgp which illustrates these issues.
> 
> Thank you for the test file.  I'll attach a patch (against
> the latest CVS) to fix those problems.

I tried this patch, but most characters are shown as whitebox.
I'm not sure but this patch fixes this problem.

--- parse.c.orig	2005-05-14 01:36:18.757842962 +0900
+++ parse.c	2005-05-14 01:32:21.681407252 +0900
@@ -982,13 +982,13 @@
 	if (! (mgp_flag & FL_NOM17N)) {
 		tmpstr[6] = ctlalloc1(CTL_M17N);
 		tmpstr[6]->ctc2_value1 = strdup("fontset");
-		tmpstr[6]->ctc2_value2 = NULL;
+		tmpstr[6]->ctc2_value2 = strdup("default");
 		tmpstr[7] = ctlalloc1(CTL_M17N);
 		tmpstr[7]->ctc2_value1 = strdup("family");
-		tmpstr[7]->ctc2_value2 = NULL;
+		tmpstr[7]->ctc2_value2 = strdup("default");
 		tmpstr[8] = ctlalloc1(CTL_M17N);
 		tmpstr[8]->ctc2_value1 = strdup("language");
-		tmpstr[8]->ctc2_value2 = NULL;
+		tmpstr[8]->ctc2_value2 = strdup("default");
 	} else
 #endif
 	{

Regards,
Fumitoshi UKAI