> Here is the patch for cygwin win32 clipboard support. > > I had to manually do the following to get it to compile: > > 1) run ./configure > 2) Add the following line to auto/config.h > > #define FEAT_CYGWIN_WIN32_CLIPBOARD 1 > > > > 3) auto/config.mk > > +OS_EXTRA_SRC = os_mswin.c > +OS_EXTRA_OBJ = objects/os_mswin.o > > > I was having troubles with the autoconf stuff. I have never used it > before, but am working on it. Also there are issues with compiling as > TINY with the clipboard support that I'm looking into.
Building a binary with both os_unix.c and os_mswin.c sounds like a bad idea. The big #ifdefs are quite a hack. On the other hand, moving this to a generic file, such as ui.c, also doesn't seem a good plan. How about moving this code to a new file, e.g., winclip.c? Including windows.h in os_unix.c looks like a mistake. There should not be windows code in a Unix-specific file. You probably have to move some code elsewhere. Probably the same winclip.c file. Instead of: #if defined (WIN3264) Please use: #if defined(WIN3264) Perhaps FEAT_CYGWIN_WIN32_CLIPBOARD can be defined automatically when building a non-GUI version with Cygwin? About calling clip_init(FALSE): see the call in os_win32.c. The one in term.c is for xterm. Instead of copying this code with the magic names it should be in a common place. Hmm, it appears it has already been duplicated... -- I'm not familiar with this proof, but I'm aware of a significant following of toddlers who believe that peanut butter is the solution to all of life's problems... -- Tim Hammerquist /// Bram Moolenaar -- [EMAIL PROTECTED] -- 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 ///