GNU bug report logs - #31288
27.0.50; [PATCH] Make cairo build somewhat usable

Previous Next

Package: emacs;

Reported by: Ari Roponen <ari.roponen <at> gmail.com>

Date: Fri, 27 Apr 2018 13:20:02 UTC

Severity: normal

Tags: patch

Found in version 27.0.50

Done: YAMAMOTO Mitsuharu <mituharu <at> math.s.chiba-u.ac.jp>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Eli Zaretskii <eliz <at> gnu.org>
To: Ari Roponen <ari.roponen <at> gmail.com>
Cc: 31288 <at> debbugs.gnu.org
Subject: bug#31288: 27.0.50; [PATCH] Make cairo build somewhat usable
Date: Fri, 27 Apr 2018 17:55:57 +0300
> From: Ari Roponen <ari.roponen <at> gmail.com>
> Cc: 31288 <at> debbugs.gnu.org
> Date: Fri, 27 Apr 2018 17:35:58 +0300
> 
> >> -#if defined (USE_GTK) && defined (HAVE_GTK3)
> >> +      /* FIXME */
> >> +#if 0 && defined (USE_GTK) && defined (HAVE_GTK3)
> >>        if (FRAME_GTK_WIDGET (f))
> >>          {
> >>            GdkWindow *w = gtk_widget_get_window (FRAME_GTK_WIDGET (f));
> >
> > This hunk doesn't look right.  If the code in question causes some
> > trouble to a Cairo build, please ifdef it out only when USE_CAIRO is
> > defined, not unconditionally.  And if that still needs a FIXME
> > comment, please add some description of what needs to be fixed and
> > why.
> >
> 
> That hunk is inside #ifdef USE_CAIRO, like almost the whole body of
> x_update_end.

Ah, sorry I missed that.  Still, what is the FIXME there? why not
simply remove that part?




This bug report was last modified 6 years and 118 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.