Removing GTK1 from vim73

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

Removing GTK1 from vim73

Tony Mechelynck
Hello

Eyeball check of the diff corresponding to the latest changeset (Remove
GTK1) seems to indicate that at line 2991/old 2990/new of src/os_unix.c,

# if defined(FEAT_X11) && (!defined(FEAT_GUI_GTK) || defined(HAVE_GTK2))

can be simplified to

# ifdef FEAT_X11

since (!defined(FEAT_GUI_GTK) || defined(HAVE_GTK2)) is now always TRUE.


Best regards,
Tony.
--
CRONE:  Who sent you?
ARTHUR: The Knights Who Say Ni!
CRONE:  Aaaagh!  (she looks around in rear) No!  We have no shrubberies
here.
                  "Monty Python and the Holy Grail" PYTHON (MONTY)
PICTURES LTD

--
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php
Reply | Threaded
Open this post in threaded view
|

Re: Removing GTK1 from vim73

Bram Moolenaar

Tony Mechelynck wrote:

> Eyeball check of the diff corresponding to the latest changeset (Remove
> GTK1) seems to indicate that at line 2991/old 2990/new of src/os_unix.c,
>
> # if defined(FEAT_X11) && (!defined(FEAT_GUI_GTK) || defined(HAVE_GTK2))
>
> can be simplified to
>
> # ifdef FEAT_X11
>
> since (!defined(FEAT_GUI_GTK) || defined(HAVE_GTK2)) is now always TRUE.

Yep, thanks.

I'll change all HAVE_GTK2 to FEAT_GUI_GTK now, since they always have
the same value.

--
"Marriage is a wonderful institution...
but who wants to live in an institution?"
 - Groucho Marx

 /// Bram Moolenaar -- [hidden email] -- http://www.Moolenaar.net   \\\
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\        download, build and distribute -- http://www.A-A-P.org        ///
 \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

--
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php