all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Philipp Stephani <phst@google.com>
To: Philipp Stephani <p.stephani2@gmail.com>, emacs-devel@gnu.org
Subject: Re: [PATCH] Work around deprecation of gtk_style_context_get_background_color
Date: Tue, 03 Oct 2017 13:02:09 +0000	[thread overview]
Message-ID: <CAP-RRPvCojUsx868rtyT28eF16c-kfFiE4Zda2Y01+r69YjVKg@mail.gmail.com> (raw)
In-Reply-To: <20170929215723.30791-1-phst@google.com>

[-- Attachment #1: Type: text/plain, Size: 1962 bytes --]

Philipp Stephani <p.stephani2@gmail.com> schrieb am Fr., 29. Sep. 2017 um
23:58 Uhr:

> * src/gtkutil.c (xg_check_special_colors): Replace call to
> gtk_style_context_get_background_color with its definition.
> ---
>  src/gtkutil.c | 17 ++++++++++++-----
>  1 file changed, 12 insertions(+), 5 deletions(-)
>
> diff --git a/src/gtkutil.c b/src/gtkutil.c
> index 9f05524738..015456cd81 100644
> --- a/src/gtkutil.c
> +++ b/src/gtkutil.c
> @@ -577,11 +577,18 @@ xg_check_special_colors (struct frame *f,
>      if (get_fg)
>        gtk_style_context_get_color (gsty, state, &col);
>      else
> -      /* FIXME: gtk_style_context_get_background_color is deprecated
> -         in GTK+ 3.16.  New versions of GTK+ don’t use the concept of
> -         a single background color any more, so we shouldn’t query for
> -         it.  */
> -      gtk_style_context_get_background_color (gsty, state, &col);
> +      {
> +        GdkRGBA *c;
> +        /* FIXME: Retrieving the background color is deprecated in
> +           GTK+ 3.16.  New versions of GTK+ don’t use the concept of a
> +           single background color any more, so we shouldn’t query for
> +           it.  */
> +        gtk_style_context_get (gsty, state,
> +                               GTK_STYLE_PROPERTY_BACKGROUND_COLOR, &c,
> +                               NULL);
> +        col = *c;
> +        gdk_rgba_free (c);
> +      }
>
>
Pushed as 0fff900c18 to emacs-26.
-- 

Google Germany GmbH
Erika-Mann-Straße 33
80636 München

Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg
Geschäftsführer: Paul Manicle, Halimah DeLaine Prado

If you received this communication by mistake, please don't forward it to
anyone else (it may contain confidential or privileged information), please
erase all copies of it, including all attachments, and please let the
sender know it went to the wrong person. Thanks.

[-- Attachment #2: Type: text/html, Size: 2598 bytes --]

  reply	other threads:[~2017-10-03 13:02 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-09-28 20:05 bug#28636: 27.0.50; Unable to load color "gtk_selection_bg_color" in 27.0.50 aplanas
2017-09-28 20:22 ` Philipp Stephani
2017-09-29  8:34   ` martin rudalics
2017-09-29 20:42     ` Philipp Stephani
2017-09-29 21:05       ` Eli Zaretskii
2017-09-29 21:42         ` Philipp Stephani
2017-09-29 21:57           ` [PATCH] Work around deprecation of gtk_style_context_get_background_color Philipp Stephani
2017-10-03 13:02             ` Philipp Stephani [this message]
2017-09-30  6:50           ` bug#28636: 27.0.50; Unable to load color "gtk_selection_bg_color" in 27.0.50 Eli Zaretskii
2017-09-29 20:44   ` Philipp Stephani

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAP-RRPvCojUsx868rtyT28eF16c-kfFiE4Zda2Y01+r69YjVKg@mail.gmail.com \
    --to=phst@google.com \
    --cc=emacs-devel@gnu.org \
    --cc=p.stephani2@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.