Don't warn if DISPLAY is not set

Don't warn if DISPLAY is not set

This is normal under Wayland, and not worth warning about.
The warnings disrupt unit tests in GNOME continuous, which
treat warnings as fatal.

BUG=https://github.com/ibus/ibus/pull/1844
R=shawn.p.huang@gmail.com

Committed: 0432aa66b8728bc266da3c2cca84587bc44b3557

[shawn.p.huang_gmail.com, 2016-03-07 13:39:18]

lgtm

On Mon, Mar 7, 2016, 07:08 <takao.fujiwara1@gmail.com> wrote:

> Reviewers: shawn.p.huang_gmail.com,
>
> Description:
> Don't warn if DISPLAY is not set
>
> This is normal under Wayland, and not worth warning about.
> The warnings disrupt unit tests in GNOME continuous, which
> treat warnings as fatal.
>
> BUG=https://github.com/ibus/ibus/pull/1844
>
> Please review this at https://codereview.appspot.com/289430043/
>
> Affected files (+1, -4 lines):
>    M src/ibusshare.c
>
>
> Index: src/ibusshare.c
> diff --git a/src/ibusshare.c b/src/ibusshare.c
> index
>
>
63800a60ba22cf9792232e275da9cb0823dda17b..b793a962f933c086a99cfc5c9dbd4001125b923e
> 100644
> --- a/src/ibusshare.c
> +++ b/src/ibusshare.c
> @@ -113,10 +113,7 @@ ibus_get_socket_path (void)
>               display = g_strdup (_display);
>           }
>
> -        if (display == NULL) {
> -            g_warning ("DISPLAY is empty! We use default DISPLAY (:0.0)");
> -        }
> -        else {
> +        if (display) {
>               p = display;
>               hostname = display;
>               for (; *p != ':' && *p != '\0'; p++);
>
>
>

[fujiwara, 2016-03-08 02:16:38]

Committed patchset #1 (id:1) manually as
0432aa66b8728bc266da3c2cca84587bc44b3557 (presubmit successful).