[PATCH] Add clipboard support for running without GUI in Mac OS X

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

Re: Add clipboard support for running without GUI in Mac OS X

Bram Moolenaar

Björn Winckler wrote:

> >> On 14 July 2010 14:27, Bram Moolenaar wrote:
> >> >
> >> > Looking at os_macosx.m, it appears these functions are not used:
> >> >        mch_gettext()
> >> >        mch_bindtextdomain()
> >> >        mch_textdomain()
> >> >
> >> > The whole section with #ifdef USE_MCH_GETTEXT  appears to be unused.
> >>
> >> I have no idea what this has been used for (if anything).  I'd say it
> >> is safe to delete (I started working on localizing MacVim a while ago
> >> and for that patch I would not have any use for these functions).
> >
> > OK.  And how about the section inside #ifdef MACOS_X_ICONVEMU ?
> > I don't see it defined anywhere.
>
> I can't see what it's used for either.  Is iconv not present on all
> versions of Mac OS X perhaps?  That sounds unlikely -- my guess is
> that this is a leftover from pre-OS X.  (A quick search did not reveal
> anything on this...I could find a man page for iconv from OS X 10.3
> but that was it.)
>
> I deleted all the leftovers (as well as a TRACE macro from os_mac.h
> that nobody should use anyway) and tried compiling with different
> versions and (as expected) there were no problems.  I've attached a
> patch that also moves the "VimPboardType" variable inside #ifdef
> FEAT_CLIPBOARD, since it is only used there.  I for one would be happy
> to see this dead code removed.

Thanks.  I'll include this.

--
GUARD #2:  Wait a minute -- supposing two swallows carried it together?
GUARD #1:  No, they'd have to have it on a line.
GUARD #2:  Well, simple!  They'd just use a standard creeper!
GUARD #1:  What, held under the dorsal guiding feathers?
GUARD #2:  Well, why not?
                                  The Quest for the Holy Grail (Monty Python)

 /// 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
12