Re: Patch 7.4.2137

2016-08-01 Fir de Conversatie Ike Devolder 
On Mon, Aug 01, 2016 at 04:51:27PM +0200, Bram Moolenaar wrote:
> 
> Ike Devolder wrote:
> 
> > On Mon, Aug 01, 2016 at 03:41:08PM +0200, Bram Moolenaar wrote:
> > > 
> > > Patch 7.4.2137
> > > Problem:Using function() with a name will find another function when 
> > > it is
> > > redefined.
> > > Solution:   Add funcref().  Refer to lambda using a partial.  Fix several
> > > reference counting issues.
> > > Files:  src/vim.h, src/structs.h, src/userfunc.c, src/eval.c,
> > > src/evalfunc.c, src/channel.c, src/proto/eval.pro,
> > > src/proto/userfunc.pro, src/if_mzsch.c, src/regexp.c, 
> > > src/misc2.c,
> > > src/if_py_both.h, src/testdir/test_lambda.vim,
> > > src/testdir/test_expr.vim, runtime/doc/eval.txt
> > > 
> > 
> > Vim does not build with tiny freatures after this patch.
> 
> Should be fixed by 7.4.2140.
> 

Yes it does build fine now. Thanks.

-- 
Ike

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

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


signature.asc
Description: PGP signature

Re: Patch 7.4.2137

2016-08-01 Fir de Conversatie Ike Devolder 
On Mon, Aug 01, 2016 at 03:41:08PM +0200, Bram Moolenaar wrote:
> 
> Patch 7.4.2137
> Problem:Using function() with a name will find another function when it is
> redefined.
> Solution:   Add funcref().  Refer to lambda using a partial.  Fix several
> reference counting issues.
> Files:  src/vim.h, src/structs.h, src/userfunc.c, src/eval.c,
> src/evalfunc.c, src/channel.c, src/proto/eval.pro,
> src/proto/userfunc.pro, src/if_mzsch.c, src/regexp.c, src/misc2.c,
> src/if_py_both.h, src/testdir/test_lambda.vim,
> src/testdir/test_expr.vim, runtime/doc/eval.txt
> 

Vim does not build with tiny freatures after this patch.

Added the output below:

./configure --prefix=/usr --localstatedir=/var/lib/vim \
--mandir=/usr/share/man --with-compiledby=BlackEagle \
--with-features=tiny --disable-gpm --enable-acl --with-x=no \
--disable-gui --enable-multibyte --disable-cscope \
--disable-netbeans --disable-perlinterp --disable-pythoninterp \
--disable-rubyinterp --enable-luainterp=no
make

checking whether make sets $(MAKE)... yes
checking for gcc... gcc
checking whether the C compiler works... yes
checking for C compiler default output file name... a.out
checking for suffix of executables... 
checking whether we are cross compiling... no
checking for suffix of object files... o
checking whether we are using the GNU C compiler... yes
checking whether gcc accepts -g... yes
checking for gcc option to accept ISO C89... none needed
checking how to run the C preprocessor... gcc -E
checking for grep that handles long lines and -e... /usr/bin/grep
checking for egrep... /usr/bin/grep -E
checking for fgrep... /usr/bin/grep -F
checking for library containing strerror... none required
checking for gawk... gawk
checking for strip... strip
checking for ANSI C header files... yes
checking for sys/wait.h that is POSIX.1 compatible... yes
checking --enable-fail-if-missing argument... no
checking for recent clang version... no
configure: checking for buggy tools...
- sh is 'GNU bash, version 4.3.46(1)-release (x86_64-unknown-linux-gnu)
Copyright (C) 2013 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later 

This is free software; you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.' - probably OK.
checking for BeOS... no
checking for QNX... no
checking for Darwin (Mac OS X)... no
checking AvailabilityMacros.h usability... no
checking AvailabilityMacros.h presence... no
checking for AvailabilityMacros.h... no
checking --with-local-dir argument... Defaulting to /usr/local
checking --with-vim-name argument... Defaulting to vim
checking --with-ex-name argument... Defaulting to ex
checking --with-view-name argument... Defaulting to view
checking --with-global-runtime argument... no
checking --with-modified-by argument... no
checking if character set is EBCDIC... no
checking --disable-smack argument... no
checking linux/xattr.h usability... yes
checking linux/xattr.h presence... yes
checking for linux/xattr.h... yes
checking attr/xattr.h usability... yes
checking attr/xattr.h presence... yes
checking for attr/xattr.h... yes
checking for XATTR_NAME_SMACKEXEC in linux/xattr.h... no
checking --disable-selinux argument... no
checking for is_selinux_enabled in -lselinux... no
checking --with-features argument... tiny
checking --with-compiledby argument... BlackEagle
checking --disable-xsmp argument... no
checking --disable-xsmp-interact argument... no
checking --enable-luainterp argument... no
checking --enable-mzschemeinterp argument... no
checking --enable-perlinterp argument... no
checking --enable-pythoninterp argument... no
checking --enable-python3interp argument... no
checking --enable-tclinterp argument... no
checking --enable-rubyinterp argument... no
checking --enable-cscope argument... no
checking --enable-workshop argument... no
checking --disable-netbeans argument... yes
checking --disable-channel argument... cannot use channels with tiny or small 
features
checking --enable-multibyte argument... yes
checking --enable-hangulinput argument... no
checking --enable-xim argument... defaulting to auto
checking --enable-fontset argument... no
defaulting to: don't HAVE_X11
checking --enable-gui argument... no GUI support
checking X11/SM/SMlib.h usability... yes
checking X11/SM/SMlib.h presence... yes
checking for X11/SM/SMlib.h... yes
checking for CYGWIN or MSYS environment... no
checking whether toupper is broken... no
checking whether __DATE__ and __TIME__ work... yes
checking whether __attribute__((unused)) is allowed... yes
checking elf.h usability... yes
checking elf.h presence... yes
checking for elf.h... yes
checking for main in -lelf... yes
checking for dirent.h that defines DIR... yes
checking for library containing opendir... none required
checking stdint.h usability... yes
checking stdint.h presence... yes
checking for stdint.h...

Re: Patch 7.4.856

2015-09-09 Fir de Conversatie Ike Devolder 
On Wed, Sep 09, 2015 at 01:38:18AM -0700, Tony Mechelynck wrote:
> On Wednesday, September 9, 2015 at 9:16:00 AM UTC+2, Ike wrote:
> > tiny build breaks with this patch, I've opened a pull request on github
> > that only uses plines_nofill when FEAT_DIFF or PROTO is defined.
> > 
> > https://github.com/vim/vim/pull/416
> > 
> > -- 
> > Ike
> 
> I think "FEAT_DIFF or PROTO" isn't enough. My Tiny build has PROTO but not 
> FEAT_DIFF and it doesn't link without the patch in the thread I started about 
> build errors. That patch has just #ifdef FEAT_DIFF ... #else ... #endif in 
> move.c plus other (quickfix-related) changes in ex_docmd.c
> 
> 
> Best regards,
> Tony.
> 
> -- 
> -- 
> 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
> 
> --- 
> You received this message because you are subscribed to the Google Groups 
> "vim_dev" group.
> To unsubscribe from this group and stop receiving emails from it, send an 
> email to vim_dev+unsubscr...@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.

the definition of plines_nofill in src/misc1.c has the same #if so when
inside:

#if defined(FEAT_DIFF) || defined(PROTO)

plines_nofill should be available

-- 
Ike

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

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


pgpoFtBM9cbgP.pgp
Description: PGP signature

Re: Warnings and errors after adding patches 855 to 861 (incl. fatal error in Tiny build)

2015-09-09 Fir de Conversatie Ike Devolder 
On Tue, Sep 08, 2015 at 08:30:22PM -0700, h_east wrote:
> Hi Tony and Yegappan!
> 
> 2015-9-9(Wed) 11:24:19 UTC+9 Tony Mechelynck:
> > On Wednesday, September 9, 2015 at 3:48:26 AM UTC+2, yegapp...@gmail.com 
> > wrote:
> > > Hi,
> > > 
> > > The patch to fix the build problems with the tiny build is attached.
> > > 
> > > - Yegappan
> > > 
> > 
> > With this patch, the tiny build succeeds, with the following warning:
> > 
> > gcc -c -I. -Iproto -DHAVE_CONFIG_H -O2 -fno-strength-reduce -Wall 
> > -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1-o objects/move.o move.c
> > move.c: In function ‘scroll_cursor_top’:
> > move.c:1775:7: warning: ‘i’ may be used uninitialized in this function 
> > [-Wmaybe-uninitialized]
> >   used += i;
> >^
> 
> I think src/move.c may be modified as below.
> What do you think?
> 
> 
> diff -r f717d96a39b3 src/move.c
> --- a/src/move.c  Tue Sep 08 23:45:05 2015 +0200
> +++ b/src/move.c  Wed Sep 09 12:23:54 2015 +0900
> @@ -1769,7 +1769,11 @@
>   i = 1;
>   else
>  #endif
> +#ifdef FEAT_DIFF
>   i = plines_nofill(top);
> +#else
> + i = plines(top);
> +#endif
>   used += i;
>   if (extra + i <= off && bot < curbuf->b_ml.ml_line_count)
>   {
> 
> 
> 
> 
> --
> Best regards,
> Hirohito Higashi (a.k.a h_east)
> 
> -- 
> -- 
> 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
> 
> --- 
> You received this message because you are subscribed to the Google Groups 
> "vim_dev" group.
> To unsubscribe from this group and stop receiving emails from it, send an 
> email to vim_dev+unsubscr...@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.

oops, only reading this thread now, to fix the tiny build fail after 856
I've also created a PR #416. Anyhow, also thanks for the fix for the
tiny build fail introduced by 858.

-- 
Ike

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

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


pgpVP79bdriXJ.pgp
Description: PGP signature

Re: Patch 7.4.856

2015-09-09 Fir de Conversatie Ike Devolder 
On Tue, Sep 08, 2015 at 05:31:53PM +0200, Bram Moolenaar wrote:
> 
> Patch 7.4.856
> Problem:"zt" still doesn't work well with filler lines. (Gary Johnson)
> Solution:   Check for filler lines above the cursor. (Christian Brabandt)
> Files:  src/move.c
> 
> 
> *** ../vim-7.4.855/src/move.c 2015-09-01 20:53:20.294030821 +0200
> --- src/move.c2015-09-08 17:18:09.315751849 +0200
> ***
> *** 1751,1762 
>   new_topline = top + 1;
>   
>   #ifdef FEAT_DIFF
> ! /* used already contains the number of filler lines above, don't add it
>* again.
> !  * TODO: if filler lines above new top are to be considered as context 
> for
> !  * the current window, leave next statement commented, else hide filler
> !  * lines above cursor line, by adding them to extra */
> ! /* extra += diff_check_fill(curwin, curwin->w_cursor.lnum); */
>   #endif
>   
>   /*
> --- 1751,1760 
>   new_topline = top + 1;
>   
>   #ifdef FEAT_DIFF
> ! /* "used" already contains the number of filler lines above, don't add 
> it
>* again.
> !  * Hide filler lines above cursor line by adding them to "extra". */
> ! extra += diff_check_fill(curwin, curwin->w_cursor.lnum);
>   #endif
>   
>   /*
> ***
> *** 1771,1777 
>   i = 1;
>   else
>   #endif
> ! i = plines(top);
>   used += i;
>   if (extra + i <= off && bot < curbuf->b_ml.ml_line_count)
>   {
> --- 1769,1775 
>   i = 1;
>   else
>   #endif
> ! i = plines_nofill(top);
>   used += i;
>   if (extra + i <= off && bot < curbuf->b_ml.ml_line_count)
>   {
> *** ../vim-7.4.855/src/version.c  2015-09-08 16:31:01.673123014 +0200
> --- src/version.c 2015-09-08 17:15:18.005529911 +0200
> ***
> *** 743,744 
> --- 743,746 
>   {   /* Add new patch number below this line */
> + /**/
> + 856,
>   /**/
> 
> -- 
> I used to wonder about the meaning of life.  But I looked it
> up in the dictionary under "L" and there it was - the meaning
> of life.  It was less than I expected.  - Dogbert
> 
>  /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net   \\\
> ///sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
> \\\  an exciting new programming language -- http://www.Zimbu.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
> 
> --- 
> You received this message because you are subscribed to the Google Groups 
> "vim_dev" group.
> To unsubscribe from this group and stop receiving emails from it, send an 
> email to vim_dev+unsubscr...@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.

tiny build breaks with this patch, I've opened a pull request on github
that only uses plines_nofill when FEAT_DIFF or PROTO is defined.

https://github.com/vim/vim/pull/416

-- 
Ike

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

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


pgp2KJRYQLVhN.pgp
Description: PGP signature

Re: Patch 7.4.856

2015-09-09 Fir de Conversatie Ike Devolder 
On Wed, Sep 09, 2015 at 03:24:00AM -0700, h_east wrote:
> Hi again,
> 
> 2015-9-9(Wed) 19:01:36 UTC+9 h_east:
> > Hi!
> > 
> > 2015-9-9(Wed) 18:17:20 UTC+9 Christian Brabandt:
> > > Am 2015-09-09 10:47, schrieb Ike Devolder:
> > > > On Wed, Sep 09, 2015 at 01:38:18AM -0700, Tony Mechelynck wrote:
> > > >> On Wednesday, September 9, 2015 at 9:16:00 AM UTC+2, Ike wrote:
> > > >> > tiny build breaks with this patch, I've opened a pull request on 
> > > >> > github
> > > >> > that only uses plines_nofill when FEAT_DIFF or PROTO is defined.
> > > >> >
> > > >> > https://github.com/vim/vim/pull/416
> > > 
> > > Sorry for causing this.
> > > 
> > > > the definition of plines_nofill in src/misc1.c has the same #if so when
> > > > inside:
> > > > 
> > > > #if defined(FEAT_DIFF) || defined(PROTO)
> > > > 
> > > > plines_nofill should be available
> > > 
> > > correct and that compiles here fine with a tiny version.
> > 
> > defined(PROTO) is no need function caller part.
> > Only global function body part need it for cproro command.
> 
> PS
> It is not to be used alone.
> Always used with the other FEAT_*.
> 
> --
> Best regards,
> Hirohito Higashi (a.k.a h_east)
> 
> -- 
> -- 
> 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
> 
> --- 
> You received this message because you are subscribed to the Google Groups 
> "vim_dev" group.
> To unsubscribe from this group and stop receiving emails from it, send an 
> email to vim_dev+unsubscr...@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.

ok so on the caller part just use #if defined(FEAT_DIFF) is sufficient

-- 
Ike

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

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


pgpaoJzUyqcq5.pgp
Description: PGP signature

Re: Patch 7.4.732

2015-06-10 Fir de Conversatie Ike Devolder 
On Wed, Jun 10, 2015 at 12:13:21AM -0700, h_east wrote:
 Hi Ike and Bram,
 
 2015/6/10(Wed) 15:51:55 UTC+9 Ike:
  On Tue, Jun 09, 2015 at 07:24:37PM +0200, Bram Moolenaar wrote:
   
   Patch 7.4.732
   Problem:The cursor line is not always updated for the O command.
   Solution:   Reset the VALID_CROW flag. (Christian Brabandt)
   Files:  src/normal.c
   
   
   *** ../vim-7.4.731/src/normal.c   2015-03-31 17:46:16.844128018 +0200
   --- src/normal.c  2015-06-09 19:18:30.394588238 +0200
   ***
   *** 8493,8498 
   --- 8493,8501 
 /* When '#' is in 'cpoptions' ignore the count. */
 if (vim_strchr(p_cpo, CPO_HASH) != NULL)
 cap-count1 = 1;
   + if (curwin-w_p_cul)
   + /* force redraw of cursorline */
   + curwin-w_valid = ~VALID_CROW;
 invoke_edit(cap, FALSE, cap-cmdchar, TRUE);
 }
 }
   *** ../vim-7.4.731/src/version.c  2015-06-09 19:14:18.777373918 +0200
   --- src/version.c 2015-06-09 19:20:00.357591407 +0200
   ***
   *** 743,744 
   --- 743,746 
 {   /* Add new patch number below this line */
   + /**/
   + 732,
 /**/
   
   -- 
   From know your smileys:
8-O  Omigod!! (done rm -rf * ?)
   
/// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net   
   \\\
   ///sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ 
   \\\
   \\\  an exciting new programming language -- http://www.Zimbu.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
   
   --- 
   You received this message because you are subscribed to the Google Groups 
   vim_dev group.
   To unsubscribe from this group and stop receiving emails from it, send an 
   email to vim_dev+unsubscr...@googlegroups.com.
   For more options, visit https://groups.google.com/d/optout.
  
  Tiny build fails after this patch:
  
  
  normal.c: In function 'n_opencmd':
  normal.c:8490:16: error: 'win_T {aka struct window_S}' has no member named 
  'w_p_cul'
if (curwin-w_p_cul)
  ^
  Makefile:2703: recipe for target 'objects/normal.o' failed
  
  Did not check any further yet.
 
 I attached a patch that fixes problem.
 
 --
 Best regards,
 Hirohito Higashi (a.k.a. h_east)
 
 -- 
 -- 
 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
 
 --- 
 You received this message because you are subscribed to the Google Groups 
 vim_dev group.
 To unsubscribe from this group and stop receiving emails from it, send an 
 email to vim_dev+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/d/optout.

 diff -r 7a8dcf699b22 src/normal.c
 --- a/src/normal.cTue Jun 09 21:33:31 2015 +0200
 +++ b/src/normal.cWed Jun 10 16:07:28 2015 +0900
 @@ -8487,9 +8487,11 @@
   /* When '#' is in 'cpoptions' ignore the count. */
   if (vim_strchr(p_cpo, CPO_HASH) != NULL)
   cap-count1 = 1;
 +#ifdef FEAT_SYN_HL
   if (curwin-w_p_cul)
   /* force redraw of cursorline */
   curwin-w_valid = ~VALID_CROW;
 +#endif
   invoke_edit(cap, FALSE, cap-cmdchar, TRUE);
   }
  }

Thanks, I confirm that this fixes it.

-- 
Ike

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

--- 
You received this message because you are subscribed to the Google Groups 
vim_dev group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


pgpcJqVLsxdvU.pgp
Description: PGP signature

Re: Patch 7.4.732

2015-06-10 Fir de Conversatie Ike Devolder 
On Tue, Jun 09, 2015 at 07:24:37PM +0200, Bram Moolenaar wrote:
 
 Patch 7.4.732
 Problem:The cursor line is not always updated for the O command.
 Solution:   Reset the VALID_CROW flag. (Christian Brabandt)
 Files:  src/normal.c
 
 
 *** ../vim-7.4.731/src/normal.c   2015-03-31 17:46:16.844128018 +0200
 --- src/normal.c  2015-06-09 19:18:30.394588238 +0200
 ***
 *** 8493,8498 
 --- 8493,8501 
   /* When '#' is in 'cpoptions' ignore the count. */
   if (vim_strchr(p_cpo, CPO_HASH) != NULL)
   cap-count1 = 1;
 + if (curwin-w_p_cul)
 + /* force redraw of cursorline */
 + curwin-w_valid = ~VALID_CROW;
   invoke_edit(cap, FALSE, cap-cmdchar, TRUE);
   }
   }
 *** ../vim-7.4.731/src/version.c  2015-06-09 19:14:18.777373918 +0200
 --- src/version.c 2015-06-09 19:20:00.357591407 +0200
 ***
 *** 743,744 
 --- 743,746 
   {   /* Add new patch number below this line */
 + /**/
 + 732,
   /**/
 
 -- 
 From know your smileys:
  8-O  Omigod!! (done rm -rf * ?)
 
  /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net   \\\
 ///sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
 \\\  an exciting new programming language -- http://www.Zimbu.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
 
 --- 
 You received this message because you are subscribed to the Google Groups 
 vim_dev group.
 To unsubscribe from this group and stop receiving emails from it, send an 
 email to vim_dev+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/d/optout.

Tiny build fails after this patch:


normal.c: In function 'n_opencmd':
normal.c:8490:16: error: 'win_T {aka struct window_S}' has no member named 
'w_p_cul'
  if (curwin-w_p_cul)
^
Makefile:2703: recipe for target 'objects/normal.o' failed

Did not check any further yet.

-- 
Ike

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

--- 
You received this message because you are subscribed to the Google Groups 
vim_dev group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


pgpyUIwpNAXAt.pgp
Description: PGP signature

Re: Patch 7.4.592

2015-01-27 Fir de Conversatie Ike Devolder 
On Tue, Jan 27, 2015 at 11:26:39AM +0100, Bram Moolenaar wrote:
 
 Patch 7.4.592
 Problem:When doing :e foobar when already editing foobar and 'buftype'
   is nofile the buffer is cleared. (Xavier de Gaye)
 Solution:   Do no clear the buffer.
 Files:src/ex_cmds.c
 
 
 *** ../vim-7.4.591/src/ex_cmds.c  2014-12-13 03:17:07.461046575 +0100
 --- src/ex_cmds.c 2015-01-27 11:21:14.752434647 +0100
 ***
 *** 3529,3534 
 --- 3529,3541 
   #endif
   check_fname() == FAIL)
   goto theend;
 + 
 + /* :e foobar when already editing foobar will reload the file.
 +  * But when 'buftype' is nofile there is no file to load, so don't
 +  * do anything. */
 + if (curbuf-b_p_bt[0] == 'n'  curbuf-b_p_bt[2] == 'f')
 + goto theend;
 + 
   oldbuf = (flags  ECMD_OLDBUF);
   }
   
 *** ../vim-7.4.591/src/version.c  2015-01-22 22:41:51.864583029 +0100
 --- src/version.c 2015-01-27 11:24:32.466265106 +0100
 ***
 *** 743,744 
 --- 743,746 
   {   /* Add new patch number below this line */
 + /**/
 + 592,
   /**/
 
 -- 
 hundred-and-one symptoms of being an internet addict:
 120. You ask a friend, What's that big shiny thing? He says, It's the sun.
 
  /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net   \\\
 ///sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
 \\\  an exciting new programming language -- http://www.Zimbu.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
 
 --- 
 You received this message because you are subscribed to the Google Groups 
 vim_dev group.
 To unsubscribe from this group and stop receiving emails from it, send an 
 email to vim_dev+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/d/optout.

Tiny build fails after this patch.


gcc -c -I. -Iproto -DHAVE_CONFIG_H   -D_FORTIFY_SOURCE=2  -march=x86-64 
-mtune=generic -O2 -pipe -fstack-protector-strong --param=ssp-buffer-size=4 
-U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1   -o objects/ex_cmds.o ex_cmds.c^M
gcc -c -I. -Iproto -DHAVE_CONFIG_H   -D_FORTIFY_SOURCE=2  -march=x86-64 
-mtune=generic -O2 -pipe -fstack-protector-strong --param=ssp-buffer-size=4 
-U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1   -o objects/ex_cmds2.o ex_cmds2.c^M
ex_cmds.c: In function 'do_ecmd':^M 


 ex_cmds.c:3536:12: error: 'buf_T' has no 
member named 'b_p_bt'^M
  if (curbuf-b_p_bt[0] == 'n'  curbuf-b_p_bt[2] == 'f')^M
^^M
ex_cmds.c:3536:40: error: 'buf_T' has no member named 'b_p_bt'^M
  if (curbuf-b_p_bt[0] == 'n'  curbuf-b_p_bt[2] == 'f')^M
^^M
gcc -c -I. -Iproto -DHAVE_CONFIG_H   -D_FORTIFY_SOURCE=2  -march=x86-64 
-mtune=generic -O2 -pipe -fstack-protector-strong --param=ssp-buffer-size=4 
-U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1   -o objects/ex_docmd.o ex_docmd.c^M
Makefile:2551: recipe for target 'objects/ex_cmds.o' failed^M
make[1]: *** [objects/ex_cmds.o] Error 1^M

-- 
Ike

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

--- 
You received this message because you are subscribed to the Google Groups 
vim_dev group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


pgpkZWgFL4TUq.pgp
Description: PGP signature

Re: Patch 7.4.571

2015-01-14 Fir de Conversatie Ike Devolder 
On Wed, Jan 14, 2015 at 02:09:04PM +0100, Bram Moolenaar wrote:
 
 Patch 7.4.571 (after 7.4.569)
 Problem:Can't build with tiny features. (Ike Devolder)
 Solution:   Add #ifdef.
 Files:src/getchar.c
 
 
 *** ../vim-7.4.570/src/getchar.c  2015-01-14 12:44:38.403422124 +0100
 --- src/getchar.c 2015-01-14 14:06:14.457992406 +0100
 ***
 *** 3710,3718 
 --- 3710,3720 
   else if (*keys == Ctrl_C)
   {
   /* If CTRL-C has been unmapped, reuse it for Interrupting. */
 + #ifdef FEAT_LOCALMAP
   if (map_table == curbuf-b_maphash)
   curbuf-b_mapped_ctrl_c = ~mode;
   else
 + #endif
   mapped_ctrl_c = ~mode;
   }
   goto theend;
 ***
 *** 3750,3758 
 --- 3752,3762 
   /* If CTRL-C has been mapped, don't always use it for Interrupting. */
   if (*keys == Ctrl_C)
   {
 + #ifdef FEAT_LOCALMAP
   if (map_table == curbuf-b_maphash)
   curbuf-b_mapped_ctrl_c |= mode;
   else
 + #endif
   mapped_ctrl_c |= mode;
   }
   
 *** ../vim-7.4.570/src/version.c  2015-01-14 14:04:05.511397639 +0100
 --- src/version.c 2015-01-14 14:07:02.889464623 +0100
 ***
 *** 743,744 
 --- 743,746 
   {   /* Add new patch number below this line */
 + /**/
 + 571,
   /**/
 
 -- 
 hundred-and-one symptoms of being an internet addict:
 91. It's Saturday afternoon in the middle of May and you
 are on computer.
 
  /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net   \\\
 ///sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
 \\\  an exciting new programming language -- http://www.Zimbu.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
 
 --- 
 You received this message because you are subscribed to the Google Groups 
 vim_dev group.
 To unsubscribe from this group and stop receiving emails from it, send an 
 email to vim_dev+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/d/optout.

Thanks, tiny builds fine again.

-- 
Ike

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

--- 
You received this message because you are subscribed to the Google Groups 
vim_dev group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


pgp5ekKnLBs2R.pgp
Description: PGP signature

Re: Patch 7.4.569

2015-01-14 Fir de Conversatie Ike Devolder 
On Wed, Jan 14, 2015 at 12:44:57PM +0100, Bram Moolenaar wrote:
 
 Patch 7.4.569 (after 7.4.468)
 Problem:Having CTRL-C interrupt or not does not check the mode of the
   mapping. (Ingo Karkat)
 Solution:   Use a bitmask with the map mode. (Christian Brabandt)
 Files:src/getchar.c, src/structs.h, src/testdir/test_mapping.in,
   src/testdir/test_mapping.ok, src/ui.c, src/globals.h
 
 
 *** ../vim-7.4.568/src/getchar.c  2014-12-14 00:43:50.335749455 +0100
 --- src/getchar.c 2015-01-14 12:13:13.136016098 +0100
 ***
 *** 3708,3715 
   if (!did_it)
   retval = 2; /* no match */
   else if (*keys == Ctrl_C)
   /* If CTRL-C has been unmapped, reuse it for Interrupting. */
 ! mapped_ctrl_c = FALSE;
   goto theend;
   }
   
 --- 3708,3720 
   if (!did_it)
   retval = 2; /* no match */
   else if (*keys == Ctrl_C)
 + {
   /* If CTRL-C has been unmapped, reuse it for Interrupting. */
 ! if (map_table == curbuf-b_maphash)
 ! curbuf-b_mapped_ctrl_c = ~mode;
 ! else
 ! mapped_ctrl_c = ~mode;
 ! }
   goto theend;
   }
   
 ***
 *** 3744,3750 
   
   /* If CTRL-C has been mapped, don't always use it for Interrupting. */
   if (*keys == Ctrl_C)
 ! mapped_ctrl_c = TRUE;
   
   mp-m_keys = vim_strsave(keys);
   mp-m_str = vim_strsave(rhs);
 --- 3749,3760 
   
   /* If CTRL-C has been mapped, don't always use it for Interrupting. */
   if (*keys == Ctrl_C)
 ! {
 ! if (map_table == curbuf-b_maphash)
 ! curbuf-b_mapped_ctrl_c |= mode;
 ! else
 ! mapped_ctrl_c |= mode;
 ! }
   
   mp-m_keys = vim_strsave(keys);
   mp-m_str = vim_strsave(rhs);
 *** ../vim-7.4.568/src/structs.h  2014-09-23 15:45:04.874801055 +0200
 --- src/structs.h 2015-01-14 12:15:33.582480344 +0100
 ***
 *** 1802,1807 
 --- 1802,1808 
   cryptstate_T *b_cryptstate; /* Encryption state while reading or 
 writing
* the file. NULL when not using encryption. */
   #endif
 + int b_mapped_ctrl_c; /* modes where CTRL-C is mapped */
   
   }; /* file_buffer */
   
 *** ../vim-7.4.568/src/testdir/test_mapping.in2014-12-14 
 00:43:50.335749455 +0100
 --- src/testdir/test_mapping.in   2015-01-14 12:11:14.197316987 +0100
 ***
 *** 8,13 
 --- 8,22 
   :inoreab чкпр   vim
   GAчкпр 

 + : mapping of ctrl-c in insert mode
 + :set cpo-= cpo-=k
 + :inoremap c-c ctrl-c
 + :cnoremap c-c dummy
 + :cunmap c-c
 + GA
 + TEST2: CTRL-C | A|
 +  
 + :nunmap c-c
   
   :  langmap should not get remapped in insert mode
   :inoremap { FAIL_ilangmap
 *** ../vim-7.4.568/src/testdir/test_mapping.ok2014-12-14 
 00:43:50.335749455 +0100
 --- src/testdir/test_mapping.ok   2015-01-14 12:11:14.197316987 +0100
 ***
 *** 1,4 
 --- 1,6 
   test starts here:
   vim
 + TEST2: CTRL-C |ctrl-cA|
 + 
   +
   +
 *** ../vim-7.4.568/src/ui.c   2014-09-19 13:46:49.550399801 +0200
 --- src/ui.c  2015-01-14 12:18:23.888618642 +0100
 ***
 *** 180,186 
   
   /* ... there is no need for CTRL-C to interrupt something, don't let
* it set got_int when it was mapped. */
 ! if (mapped_ctrl_c)
   ctrl_c_interrupts = FALSE;
   }
   
 --- 180,186 
   
   /* ... there is no need for CTRL-C to interrupt something, don't let
* it set got_int when it was mapped. */
 ! if ((mapped_ctrl_c | curbuf-b_mapped_ctrl_c)  State)
   ctrl_c_interrupts = FALSE;
   }
   
 *** ../vim-7.4.568/src/globals.h  2014-08-10 13:34:59.056785459 +0200
 --- src/globals.h 2015-01-14 12:13:58.959514980 +0100
 ***
 *** 958,964 
   #ifdef USE_ON_FLY_SCROLL
   EXTERN int  dont_scroll INIT(= FALSE);/* don't use scrollbars when TRUE */
   #endif
 ! EXTERN int  mapped_ctrl_c INIT(= FALSE); /* CTRL-C is mapped */
   EXTERN int  ctrl_c_interrupts INIT(= TRUE); /* CTRL-C sets got_int */
   
   EXTERN cmdmod_T cmdmod; /* Ex command modifiers */
 --- 958,964 
   #ifdef USE_ON_FLY_SCROLL
   EXTERN int  dont_scroll INIT(= FALSE);/* don't use scrollbars when TRUE */
   #endif
 ! EXTERN int  mapped_ctrl_c INIT(= FALSE); /* modes where CTRL-C is mapped */
   EXTERN int  ctrl_c_interrupts INIT(= TRUE); /* CTRL-C sets got_int */
   
   EXTERN cmdmod_T cmdmod; /* Ex command modifiers */
 *** ../vim-7.4.568/src/version.c  2015-01-14 11:24:51.851582151 +0100
 --- src/version.c 2015-01-14 12:12:04.728764264 +0100
 ***
 *** 743,744 
 --- 743,746 
   {   /* Add new patch number below this line */
 + /**/
 + 569,
   /**/
 
 -- 
 hundred-and-one symptoms of being an internet addict:
 85. Choice between paying Compuserve bill and paying for kids

Re: Dynamic ruby 2.2.0 broken

2015-01-11 Fir de Conversatie Ike Devolder 
On Sat, Jan 10, 2015 at 09:32:41AM -0800, Ken Takata wrote:
 Hi Ike,
 
 2015/1/8 Thu 0:01:18 UTC+9 Ike wrote:
  Since version 2.2.0 of ruby building vim with dynamic ruby fails.
 ...
  if_ruby.c:(.text+0x8fa): undefined reference to `rb_check_type'
 ...
  if_ruby.c:(.text+0xbc3): undefined reference to 
  `rb_gc_writebarrier_unprotect'
 
 
 Reproduced with MSVC10.
 Can you try the attached patch?
 
 Regards,
 Ken Takata
 
 -- 
 -- 
 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
 
 --- 
 You received this message because you are subscribed to the Google Groups 
 vim_dev group.
 To unsubscribe from this group and stop receiving emails from it, send an 
 email to vim_dev+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/d/optout.

 # HG changeset patch
 # Parent 80f787a1e4be59822b1c23010a44507ed0fb2fb0
 diff --git a/src/if_ruby.c b/src/if_ruby.c
 --- a/src/if_ruby.c
 +++ b/src/if_ruby.c
 @@ -96,11 +96,15 @@
  # define rb_num2int rb_num2int_stub
  #endif
  
 -#if defined(DYNAMIC_RUBY_VER)  DYNAMIC_RUBY_VER = 21
 +#if defined(DYNAMIC_RUBY_VER)  DYNAMIC_RUBY_VER == 21
  /* Ruby 2.1 adds new GC called RGenGC and RARRAY_PTR uses
   * rb_gc_writebarrier_unprotect_promoted if USE_RGENGC  */
  # define rb_gc_writebarrier_unprotect_promoted 
 rb_gc_writebarrier_unprotect_promoted_stub
  #endif
 +#if defined(DYNAMIC_RUBY_VER)  DYNAMIC_RUBY_VER = 22
 +# define rb_gc_writebarrier_unprotect rb_gc_writebarrier_unprotect_stub
 +# define rb_check_type rb_check_type_stub
 +#endif
  
  #include ruby.h
  #ifdef RUBY19_OR_LATER
 @@ -180,7 +184,9 @@ static void ruby_vim_init(void);
   */
  # define rb_assoc_newdll_rb_assoc_new
  # define rb_cObject  (*dll_rb_cObject)
 -# define rb_check_type   dll_rb_check_type
 +# if defined(DYNAMIC_RUBY_VER)  DYNAMIC_RUBY_VER  22
 +#  define rb_check_type  dll_rb_check_type
 +# endif
  # define rb_class_path   dll_rb_class_path
  # define rb_data_object_allocdll_rb_data_object_alloc
  # define rb_define_class_under   dll_rb_define_class_under
 @@ -382,7 +388,11 @@ static void* (*ruby_process_options)(int
  # endif
  
  # if defined(USE_RGENGC)  USE_RGENGC
 +#  if defined(DYNAMIC_RUBY_VER)  DYNAMIC_RUBY_VER == 21
  static void (*dll_rb_gc_writebarrier_unprotect_promoted)(VALUE);
 +#  else
 +static void (*dll_rb_gc_writebarrier_unprotect)(VALUE obj);
 +#  endif
  # endif
  
  # if defined(RUBY19_OR_LATER)  !defined(PROTO)
 @@ -420,10 +430,24 @@ VALUE rb_num2ulong(VALUE x)
  
 /* Do not generate a prototype here, VALUE isn't always defined. */
  # if defined(USE_RGENGC)  USE_RGENGC  !defined(PROTO)
 +#  if defined(DYNAMIC_RUBY_VER)  DYNAMIC_RUBY_VER == 21
  void rb_gc_writebarrier_unprotect_promoted_stub(VALUE obj)
  {
  dll_rb_gc_writebarrier_unprotect_promoted(obj);
  }
 +#  else
 +void rb_gc_writebarrier_unprotect_stub(VALUE obj)
 +{
 +dll_rb_gc_writebarrier_unprotect(obj);
 +}
 +#  endif
 +# endif
 +
 +# if defined(DYNAMIC_RUBY_VER)  DYNAMIC_RUBY_VER = 22
 +void rb_check_type_stub(VALUE v, int i)
 +{
 +dll_rb_check_type(v, i);
 +}
  # endif
  
  static HINSTANCE hinstRuby = NULL; /* Instance of ruby.dll */
 @@ -542,7 +566,11 @@ static struct
  {ruby_init_stack, (RUBY_PROC*)dll_ruby_init_stack},
  # endif
  # if defined(USE_RGENGC)  USE_RGENGC
 +#  if defined(DYNAMIC_RUBY_VER)  DYNAMIC_RUBY_VER == 21
  {rb_gc_writebarrier_unprotect_promoted, 
 (RUBY_PROC*)dll_rb_gc_writebarrier_unprotect_promoted},
 +#  else
 +{rb_gc_writebarrier_unprotect, 
 (RUBY_PROC*)dll_rb_gc_writebarrier_unprotect},
 +#  endif
  # endif
  {, NULL},
  };

I can confirm compiling works with gcc 4.9.2 and ruby 2.2.0

simple test by running :ruby print 'test' did work when ruby is
installed.

thanks

-- 
Ike

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

--- 
You received this message because you are subscribed to the Google Groups 
vim_dev group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


pgpedpTBetIl_.pgp
Description: PGP signature

Re: Dynamic ruby 2.2.0 broken

2015-01-08 Fir de Conversatie Ike Devolder 
On Wed, Jan 07, 2015 at 11:48:30PM +0100, tux. wrote:
 Works with MSVC.
 
 -- 
 -- 
 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
 
 --- 
 You received this message because you are subscribed to the Google Groups 
 vim_dev group.
 To unsubscribe from this group and stop receiving emails from it, send an 
 email to vim_dev+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/d/optout.

with gcc 4.9.2 on linux it fails.

Currently I'm going to just build without ruby since there is always
some sort of an issue related to it.

-- 
Ike

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

--- 
You received this message because you are subscribed to the Google Groups 
vim_dev group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


pgpSg3tN8IQlo.pgp
Description: PGP signature

Dynamic ruby 2.2.0 broken

2015-01-07 Fir de Conversatie Ike Devolder 
Since version 2.2.0 of ruby building vim with dynamic ruby fails.


gcc -c -I. -Iproto -DHAVE_CONFIG_H   -D_FORTIFY_SOURCE=2  -march=x86-64
-mtune=generic -O2 -pipe -fstack-protector-strong
--param=ssp-buffer-size=4 -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1
-DDYNAMIC_RUBY_DLL=\libruby.so.2.2.0\ -DDYNAMIC_RUBY_VER=22
-I/usr/include/ruby-2.2.0 -I/usr/include/ruby-2.2.0/x86_64-linux
-DRUBY_VERSION=22 -o objects/if_ruby.o if_ruby.c

if_ruby.c:185:0: warning: rb_data_object_alloc redefined
 # define rb_data_object_alloc  dll_rb_data_object_alloc
 ^
In file included from /usr/include/ruby-2.2.0/ruby.h:33:0,
 from if_ruby.c:105:
/usr/include/ruby-2.2.0/ruby/ruby.h:1203:0: note: this is the location of the 
previous definition
 #define rb_data_object_alloc  RUBY_MACRO_SELECT(rb_data_object_alloc_, 
RUBY_UNTYPED_DATA_WARNING)
 ^
if_ruby.c:227:0: warning: rb_str_new redefined
 # define rb_str_new   dll_rb_str_new
 ^
In file included from /usr/include/ruby-2.2.0/ruby/ruby.h:1710:0,
 from /usr/include/ruby-2.2.0/ruby.h:33,
 from if_ruby.c:105:
/usr/include/ruby-2.2.0/ruby/intern.h:793:0: note: this is the location of the 
previous definition
 #define rb_str_new(str, len) __extension__ ( \
 ^
if_ruby.c:274:0: warning: rb_enc_str_new redefined
 #  define rb_enc_str_new   dll_rb_enc_str_new
 ^
In file included from if_ruby.c:107:0:
/usr/include/ruby-2.2.0/ruby/encoding.h:110:0: note: this is the location of 
the previous definition
 #define rb_enc_str_new(str, len, enc) __extension__ ( \
 ^


link.sh: $LINK_AS_NEEDED set to 'yes': invoking linker directly.
  gcc   -L. -Wl,-O1,--sort-common,--as-needed,-z,relro -fstack-protector
  -rdynamic -Wl,-export-dynamic -Wl,-E
  -Wl,-rpath,/usr/lib/perl5/core_perl/CORE
  -Wl,-O1,--sort-common,--as-needed,-z,relro -L/usr/local/lib
  -Wl,--as-needed -o vim objects/buffer.o objects/blowfish.o
  objects/charset.o objects/crypt.o objects/crypt_zip.o objects/diff.o
  objects/digraph.o objects/edit.o objects/eval.o objects/ex_cmds.o
  objects/ex_cmds2.o objects/ex_docmd.o objects/ex_eval.o
  objects/ex_getln.o objects/fileio.o objects/fold.o objects/getchar.o
  objects/hardcopy.o objects/hashtab.o  objects/if_cscope.o
  objects/if_xcmdsrv.o objects/mark.o objects/memline.o objects/menu.o
  objects/message.o objects/misc1.o objects/misc2.o objects/move.o
  objects/mbyte.o objects/normal.o objects/ops.o objects/option.o
  objects/os_unix.o objects/pathdef.o objects/popupmnu.o
  objects/quickfix.o objects/regexp.o objects/screen.o objects/search.o
  objects/sha256.o objects/spell.o objects/syntax.o  objects/tag.o
  objects/term.o objects/ui.o objects/undo.o objects/version.o
  objects/window.o  objects/if_lua.o  objects/if_perl.o
  objects/if_perlsfio.o objects/if_python.o objects/if_python3.o
  objects/if_ruby.o objects/main.o objects/memfile.o-lSM -lICE
  -lXpm -lXt -lX11 -lXdmcp -lSM -lICE  -lm -lncurses -lelf-lacl
  -lattr -lgpm -ldl   -Wl,-E -Wl,-rpath,/usr/lib/perl5/core_perl/CORE
  -Wl,-O1,--sort-common,--as-needed,-z,relro -fstack-protector
  -L/usr/local/lib  -L/usr/lib/perl5/core_perl/CORE -lperl -lnsl -ldl
  -lm -lcrypt -lutil -lpthread -lc   

objects/if_ruby.o: In function `get_win':
if_ruby.c:(.text+0x8fa): undefined reference to `rb_check_type'
objects/if_ruby.o: In function `get_buf':
if_ruby.c:(.text+0x9da): undefined reference to `rb_check_type'
objects/if_ruby.o: In function `window_set_cursor':
if_ruby.c:(.text+0xbc3): undefined reference to `rb_gc_writebarrier_unprotect'
if_ruby.c:(.text+0xbd8): undefined reference to `rb_gc_writebarrier_unprotect'
if_ruby.c:(.text+0xbe7): undefined reference to `rb_gc_writebarrier_unprotect'
if_ruby.c:(.text+0xbf8): undefined reference to `rb_gc_writebarrier_unprotect'
if_ruby.c:(.text+0xc71): undefined reference to `rb_gc_writebarrier_unprotect'
objects/if_ruby.o:if_ruby.c:(.text+0xc99): more undefined references to 
`rb_gc_writebarrier_unprotect' follow
collect2: error: ld returned 1 exit status
link.sh: Linking failed

I think the above are the relevant parts

-- 
Ike

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

--- 
You received this message because you are subscribed to the Google Groups 
vim_dev group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


pgpVZh0qRKs_f.pgp
Description: PGP signature

Re: [PATCH] Fix compilation failure with Ruby 2.1.0

2014-02-02 Fir de Conversatie Ike Devolder 
On Sun, Feb 02, 2014 at 03:34:18PM +0100, tux. wrote:
 Bram Moolenaar schrob am Sonntag, 2. Februar 2014 um 13:52 Zeit:
 
  Thanks.  I would appreciate a few people verifying this patch works fine
  with different Ruby versions.
 
 My Windows builds work fine with this alternate patch:
 https://build.opensuse.org/package/view_file/editors/vim/vim-7.4-ruby21.patch?expand=1
 
 -- 
 -- 
 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
 
 --- 
 You received this message because you are subscribed to the Google Groups 
 vim_dev group.
 To unsubscribe from this group and stop receiving emails from it, send an 
 email to vim_dev+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/groups/opt_out.

I'm also using the opensuse patch and it works fine on Archlinux.


-- 
Ike


pgpnS3huTvuys.pgp
Description: PGP signature

[vim] 7.4 very slow with syntax highlighting on (forwarded from ArchLinux bugtracker)

2013-09-26 Fir de Conversatie Ike Devolder 
This bug was reported in the Arch Linux bugtracker [1]

Just tested this behaviour in 7.4.041 and it stays the same.
With the new regexpengine the movement with j and k is massively slow.
Visible lag on a core i7 3GHz is extreme i might say.

I personally have no idea how to fix it right now so the users who reported
this problem was suggested to set regexpengine=1 in his vimrc for now.

With regexpengine set to 1 the slowness goes away.

thx if anyone wants to look into this.

[1] https://bugs.archlinux.org/task/36693

-- 
Ike

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

--- 
You received this message because you are subscribed to the Google Groups 
vim_dev group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.
call pathogen#infect()
let g:pathogen_disabled = ['vim-rails.git']

let g:Powerline_symbols = 'unicode'
set rtp+=/home/tdi/.vim/bundle/powerline/powerline/bindings/vim
syntax on


 NERD
nmap silent c-n :NERDTreeToggleCR
nmap silent F8 :TagbarToggleCR

set nocompatible
set hlsearch
 tabs and firends
set autoindent
set smartindent
set expandtab
set tabstop=2
set shiftwidth=2
set textwidth=100
set ignorecase 
set smartcase
set incsearch
set autochdir
 directory for swp files
set directory=/tmp
set wildmenu
set wildmode=longest,list
set showcmd
set showmatch
set pastetoggle=F10
filetype plugin on
set grepprg=grep\ -nH\ $*
filetype indent on 
 set mouse in terminal to resize windows
set mouse=a
set laststatus=2 

GUI
if has('gui_running')
  set guioptions-=T  remove toolbar
  set guioptions-=r  remove right-hand scroll bar
  let g:solarized_termcolors=256
  colorscheme solarized
  set background=dark
else
  if $TERM =~ '^xterm-256color'
colorscheme wombatterm
  elseif $TERM =~ '^screen'
colorscheme wombatterm
  else
colorscheme vividchalk 
  endif
endif

cmap w!! %!sudo tee  /dev/null %
tab navigation
from Practical Vim book
nnoremap silent C-h :bprevCR 
nnoremap silent C-l :bnextCR
nnoremap silent [b :bprevCR 
nnoremap silent ]b :bnextCR
nnoremap silent [B :bfirstCR
nnoremap silent ]B :blastCR

 Reselect last pasted text
nnoremap gp `[v`]
 highlight search matching
 włączenie (zp) i wyłączenie (zP) korekty pisowni dla j.polskiego
map zp :setlocal spell spelllang=plCR
map zP :setlocal nospellCR

  włączenie (ze) i wyłączenie (zE) korekty pisowni dla j.angielskiego
map ze :setlocal spell spelllang=enCR
map zE :setlocal nospellCR
map zus :setlocal spell spelllang=en_usCR

if $TERM =~ '^screen-256color'
map EscOH Home
map! EscOH Home
map EscOF End
map! EscOF End
endif

set cursorline
set cursorcolumn

fu! ToggleCurline ()
  if cursorline  cursorcolumn 
set nocursorline
set nocursorcolumn
  else
set cursorline
set cursorcolumn
  endif
endfunction

map silentleadercl :call ToggleCurline()CR

 Until I got used to it
inoremap  Up NOP
inoremap  Down   NOP
inoremap  Left   NOP
inoremap  Right  NOP
noremap   Up NOP
noremap   Down   NOP
noremap   Left   NOP
noremap   Right  NOP

imap jj Esc
nnoremap S-CR OEsc
nnoremap CR oEsc
map EscOM S-CR
 Python
let g:jedi#auto_initialization = 0
let g:jedi#auto_vim_configuration = 1
let g:jedi#show_call_signatures = 0
let g:jedi#popup_on_dot = 0

 


seeds.rb
Description: application/ruby

Re: Problem with the latest changes to syntax/css.vim

2013-09-23 Fir de Conversatie Ike Devolder 
Op maandag 23 september 2013 09:07:28 schreef Kent Sibilev:
 Hi everyone!
 
 The latest revision of syntax/css.vim introduced a subtle problem with the
 following line:
 
  Required for cssHacks
 setlocal iskeyword-=_
 
 I'm not sure if the syntax file that is the correct place to set vim's
 options other than syntax settings. For example, rails.vim plugin allows
 highlighting of xhtml syntax inside of ruby files, by loading syntax/xhtml
 and syntax/css.vim for ruby files. By setting iskeyword setting in the
 syntax file, the word completion is broken for any filetype that happens to
 be reusing css syntax.
 
 I believe that 5354 revision has introduced this change:
 
 hg diff -r5354 runtime/syntax/css.vim
 
 Regards,
 Kent.

everything that supports mixed highlighting related to html suffers from that 
line. php syntax is also broken because of the above stated line.

-- 
Ike

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

--- 
You received this message because you are subscribed to the Google Groups 
vim_dev group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.

dynamic perl fails

2013-05-27 Fir de Conversatie Ike Devolder 
small snippet of the failure when building with dynamic perl and perl is
updated to 5.18.0:

if_perl.c: In function 'XS_VIM_Msg':
if_perl.c:1118:8: warning: passing argument 1 of
'dll_Perl_croak_xs_usage' from incompatible pointer type [enabled by
default]
if_perl.c:1118:8: note: expected 'struct PerlInterpreter *' but argument
is of type 'struct CV *'
if_perl.c:1118:8: warning: passing argument 2 of
'dll_Perl_croak_xs_usage' from incompatible pointer type [enabled by
default]
if_perl.c:1118:8: note: expected 'const struct CV * const' but argument
is of type 'char *'
if_perl.c:1118:8: error: too few arguments to function
'dll_Perl_croak_xs_usage'

from perl 5.18.0 onwards the trick for croak_xs_usage seems no longer
nescessary (i could be mistaken)

patch attached.
-- 
Ike

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

--- 
You received this message because you are subscribed to the Google Groups 
vim_dev group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.


diff -r f7944ce459ac src/if_perl.xs
--- a/src/if_perl.xsSat May 25 23:15:27 2013 +0200
+++ b/src/if_perl.xsMon May 27 12:17:49 2013 +0200
@@ -67,6 +67,10 @@
 # define PERL5101_OR_LATER
 #endif
 
+#if (PERL_REVISION == 5)  (PERL_VERSION = 18)
+# define PERL5180_OR_LATER
+#endif
+
 #ifndef pTHX
 #define pTHX void
 #define pTHX_
@@ -127,9 +131,11 @@
 # define perl_free dll_perl_free
 # define Perl_get_context dll_Perl_get_context
 # define Perl_croak dll_Perl_croak
+# ifndef PERL5180_OR_LATER
 # ifdef PERL5101_OR_LATER
 #  define Perl_croak_xs_usage dll_Perl_croak_xs_usage
 # endif
+# endif
 # ifndef PROTO
 #  define Perl_croak_nocontext dll_Perl_croak_nocontext
 #  define Perl_call_argv dll_Perl_call_argv
@@ -242,9 +248,11 @@
 static int (*perl_parse)(PerlInterpreter*, XSINIT_t, int, char**, char**);
 static void* (*Perl_get_context)(void);
 static void (*Perl_croak)(pTHX_ const char*, ...);
+#ifndef PERL5180_OR_LATER
 #ifdef PERL5101_OR_LATER
 static void (*Perl_croak_xs_usage)(pTHX_ const CV *const, const char *const 
params);
 #endif
+#endif
 static void (*Perl_croak_nocontext)(const char*, ...);
 static I32 (*Perl_dowantarray)(pTHX);
 static void (*Perl_free_tmps)(pTHX);
@@ -362,9 +370,11 @@
 {perl_parse, (PERL_PROC*)perl_parse},
 {Perl_get_context, (PERL_PROC*)Perl_get_context},
 {Perl_croak, (PERL_PROC*)Perl_croak},
+#ifndef PERL5180_OR_LATER
 #ifdef PERL5101_OR_LATER
 {Perl_croak_xs_usage, (PERL_PROC*)Perl_croak_xs_usage},
 #endif
+#endif
 {Perl_croak_nocontext, (PERL_PROC*)Perl_croak_nocontext},
 {Perl_dowantarray, (PERL_PROC*)Perl_dowantarray},
 {Perl_free_tmps, (PERL_PROC*)Perl_free_tmps},

Re: Patch 7.3.864

2013-03-17 Fir de Conversatie Ike Devolder 
Op zondag 17 maart 2013 19:09:36 schreef Roland Eggner:
 On 2013-03-16 Saturday at 21:35 +0100 Bram Moolenaar wrote:
  Patch 7.3.864 (after 7.3.862)
  Problem:Can't build without the mouse feature.
  Solution:   Add an #ifdef. (Ike Devolder)
  Files:  src/misc1.c
 
 This is great, thanks.  Now vim builds with  _all_  mouse-related
 features disabled … :)
 
 
 VIM - Vi IMproved 7.3 (2010 Aug 15, compiled Mar 17 2013 12:19:55)
 Included patches: 1-865
 Modified by Gentoo-7.3.865-odvx1@systomanalyson.not_s/o/e/g
 Compiled by odvx1@systomanalyson.not_s/o/e/g
 Huge version without GUI.  Features included (+) or not (-):
 -arabic  -farsi   -mouse_netterm   +syntax
 +autocmd +file_in_path-mouse_sgr   +tag_binary
 -balloon_eval+find_in_path-mouse_sysmouse  +tag_old_static
 -browse  +float   -mouse_urxvt -tag_any_white
 ++builtin_terms  +folding -mouse_xterm -tcl
 +byte_offset -footer  +multi_byte  +terminfo
 +cindent +fork()  +multi_lang  +termresponse
 -clientserver+gettext -mzscheme+textobjects
 -clipboard   -hangul_input-netbeans_intg   +title
 +cmdline_compl   +iconv   +path_extra  -toolbar
 +cmdline_hist+insert_expand   +perl+user_commands
 +cmdline_info+jumplist-persistent_undo +vertsplit
 +comments+keymap  -printer +virtualedit
 +conceal +langmap +profile +visual
 -cryptv  +libcall +python  +visualextra
 +cscope  +linebreak   -python3 +viminfo
 +cursorbind  +lispindent  +quickfix+vreplace
 +cursorshape +listcmds+reltime +wildignore
 +dialog_con  +localmap-rightleft   +wildmenu
 +diff-lua -ruby+windows
 +digraphs+menu+scrollbind  +writebackup
 -dnd +mksession   +signs   -X11
 -ebcdic  +modify_fname+smartindent -xfontset
 +emacs_tags  -mouse   -sniff   -xim
 +eval-mouse_dec   +startuptime -xsmp
 +ex_extra-mouse_gpm   +statusline  -xterm_clipboard
 +extra_search-mouse_jsbterm   -sun_workshop-xterm_save
system vimrc file: /etc/vim/vimrc
  user vimrc file: $HOME/.vimrc
   user exrc file: $HOME/.exrc
   fall-back for $VIM: /usr/share/vim
 Compilation: x86_64-pc-linux-gnu-gcc -c -I. -Iproto -DHAVE_CONFIG_H   
  -O2 -march=native -pipe -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1
 Linking: x86_64-pc-linux-gnu-gcc   -Wl,-E  -Wl,-O1 -Wl,--as-needed
 -L/usr/local/lib -Wl,--as-needed -o vim-lm   -lcurses -ldl  
 -Wl,-E -Wl,-O1 -Wl,--as-needed 
 -L/usr/lib64/perl5/5.12.3/x86_64-linux/CORE -lperl -lnsl -ldl -lm
 -lcrypt -lutil -lc -L/usr/lib/python2.7/config -lpython2.7 -lpthread
 -ldl -lutil -lm -Xlinker -export-dynamic
 
 
 --
 Roland Eggner

I don't really have issues with the cli version here:
VIM - Vi IMproved 7.3 (2010 Aug 15, compiled Mar 16 2013 22:40:51)
Included patches: 1-865
Compiled by BlackEagle
Huge version without GUI.  Features included (+) or not (-):
+arabic  +file_in_path+mouse_sgr   +tag_binary
+autocmd +find_in_path-mouse_sysmouse  +tag_old_static
-balloon_eval+float   +mouse_urxvt -tag_any_white
-browse  +folding +mouse_xterm -tcl
++builtin_terms  -footer  +multi_byte  +terminfo
+byte_offset +fork()  +multi_lang  +termresponse
+cindent +gettext -mzscheme+textobjects
-clientserver-hangul_input-netbeans_intg   +title
-clipboard   +iconv   +path_extra  -toolbar
+cmdline_compl   +insert_expand   +perl/dyn+user_commands
+cmdline_hist+jumplist+persistent_undo +vertsplit
+cmdline_info+keymap  +postscript  +virtualedit
+comments+langmap +printer +visual
+conceal +libcall +profile +visualextra
+cryptv  +linebreak   +python/dyn  +viminfo
+cscope  +lispindent  +python3/dyn +vreplace
+cursorbind  +listcmds+quickfix+wildignore
+cursorshape +localmap+reltime +wildmenu
+dialog_con  +lua/dyn +rightleft   +windows
+diff+menu+ruby/dyn+writebackup
+digraphs+mksession   +scrollbind  -X11
-dnd +modify_fname+signs   -xfontset
-ebcdic  +mouse   +smartindent -xim
+emacs_tags  -mouseshape  -sniff   -xsmp
+eval+mouse_dec   +startuptime -xterm_clipboard
+ex_extra+mouse_gpm   +statusline  -xterm_save
+extra_search-mouse_jsbterm   -sun_workshop
+farsi   +mouse_netterm   +syntax  
   system vimrc file: /etc/vimrc
 user vimrc file: $HOME/.vimrc

Re: Patch 7.3.862

2013-03-16 Fir de Conversatie Ike Devolder 
Op zaterdag 16 maart 2013 14:21:23 schreef Bram Moolenaar:
 Patch 7.3.862
 Problem:Dragging the status line can be slow.
 Solution:   Look ahead and drop the drag event if there is a next one.
 Files:src/eval.c, src/misc1.c, src/proto/misc1.pro, 
src/normal.c
 
 
 *** ../vim-7.3.861/src/eval.c 2013-02-26 21:43:28.0 +0100
 --- src/eval.c2013-03-16 14:02:36.0 +0100
 ***
 *** 11238,11264 
   rettv-vval.v_string = vim_strsave(temp);
 
   #ifdef FEAT_MOUSE
 ! if (n == K_LEFTMOUSE
 ! || n == K_LEFTMOUSE_NM
 ! || n == K_LEFTDRAG
 ! || n == K_LEFTRELEASE
 ! || n == K_LEFTRELEASE_NM
 ! || n == K_MIDDLEMOUSE
 ! || n == K_MIDDLEDRAG
 ! || n == K_MIDDLERELEASE
 ! || n == K_RIGHTMOUSE
 ! || n == K_RIGHTDRAG
 ! || n == K_RIGHTRELEASE
 ! || n == K_X1MOUSE
 ! || n == K_X1DRAG
 ! || n == K_X1RELEASE
 ! || n == K_X2MOUSE
 ! || n == K_X2DRAG
 ! || n == K_X2RELEASE
 ! || n == K_MOUSELEFT
 ! || n == K_MOUSERIGHT
 ! || n == K_MOUSEDOWN
 ! || n == K_MOUSEUP)
   {
   int row = mouse_row;
   int col = mouse_col;
 --- 11238,11244 
   rettv-vval.v_string = vim_strsave(temp);
 
   #ifdef FEAT_MOUSE
 ! if (is_mouse_key(n))
   {
   int row = mouse_row;
   int col = mouse_col;
 *** ../vim-7.3.861/src/misc1.c2013-03-13 17:01:47.0 +0100
 --- src/misc1.c   2013-03-16 14:08:05.0 +0100
 ***
 *** 3288,3293 
 --- 3288,3325 
   return r;
   }
 
 + #if defined(FEAT_MOUSE) || defined(PROTO)
 + /*
 +  * Return TRUE if c is a mouse key.
 +  */
 + int
 + is_mouse_key(c)
 + int c;
 + {
 + return c == K_LEFTMOUSE
 + || c == K_LEFTMOUSE_NM
 + || c == K_LEFTDRAG
 + || c == K_LEFTRELEASE
 + || c == K_LEFTRELEASE_NM
 + || c == K_MIDDLEMOUSE
 + || c == K_MIDDLEDRAG
 + || c == K_MIDDLERELEASE
 + || c == K_RIGHTMOUSE
 + || c == K_RIGHTDRAG
 + || c == K_RIGHTRELEASE
 + || c == K_MOUSEDOWN
 + || c == K_MOUSEUP
 + || c == K_MOUSELEFT
 + || c == K_MOUSERIGHT
 + || c == K_X1MOUSE
 + || c == K_X1DRAG
 + || c == K_X1RELEASE
 + || c == K_X2MOUSE
 + || c == K_X2DRAG
 + || c == K_X2RELEASE;
 + }
 + #endif
 +
   /*
* Get a key stroke directly from the user.
* Ignores mouse clicks and scrollbar events, except a click for the
 left ***
 *** 3373,3403 
   n = TO_SPECIAL(buf[1], buf[2]);
   if (buf[1] == KS_MODIFIER
 
   || n == K_IGNORE
 
 ! #ifdef FEAT_MOUSE
 ! || n == K_LEFTMOUSE_NM
 ! || n == K_LEFTDRAG
 ! || n == K_LEFTRELEASE
 ! || n == K_LEFTRELEASE_NM
 ! || n == K_MIDDLEMOUSE
 ! || n == K_MIDDLEDRAG
 ! || n == K_MIDDLERELEASE
 ! || n == K_RIGHTMOUSE
 ! || n == K_RIGHTDRAG
 ! || n == K_RIGHTRELEASE
 ! || n == K_MOUSEDOWN
 ! || n == K_MOUSEUP
 ! || n == K_MOUSELEFT
 ! || n == K_MOUSERIGHT
 ! || n == K_X1MOUSE
 ! || n == K_X1DRAG
 ! || n == K_X1RELEASE
 ! || n == K_X2MOUSE
 ! || n == K_X2DRAG
 ! || n == K_X2RELEASE
 ! # ifdef FEAT_GUI
 
   || n == K_VER_SCROLLBAR
   || n == K_HOR_SCROLLBAR
 
 - # endif
   #endif
  )
   {
 --- 3405,3414 
   n = TO_SPECIAL(buf[1], buf[2]);
   if (buf[1] == KS_MODIFIER
 
   || n == K_IGNORE
 
 ! || (is_mouse_key(n)  n != K_LEFTMOUSE)
 ! #ifdef FEAT_GUI
 
   || n == K_VER_SCROLLBAR
   || n == K_HOR_SCROLLBAR
 
   #endif
  )
   {
 *** ../vim-7.3.861/src/proto/misc1.pro2012-11-20 
16:56:49.0
 +0100 --- src/proto/misc1.pro 2013-03-16 14:02:33.0 +0100
 ***
 *** 42,47 
 --- 42,48 
   void check_status __ARGS((buf_T *buf));
   void change_warning __ARGS((int col));
   int ask_yesno __ARGS((char_u *str, int direct));
 + int is_mouse_key __ARGS((int c));
   int get_keystroke __ARGS((void));
   int get_number __ARGS((int colon, int *mouse_used));
   int prompt_for_number __ARGS((int *mouse_used));
 *** ../vim-7.3.861/src/normal.c   2013-03-13 19:02:37.0 +0100
 --- src/normal.c  2013-03-16 14:09:34.0 +0100
 ***
 *** 2443,2449 
   return FALSE;
   }
 
 ! which_button = get_mouse_button(KEY2TERMCAP1(c), is_click,
 is_drag);
 
   #ifdef FEAT_MOUSESHAPE
   /* May have stopped dragging the status or

Re: Patch 7.3.737

2012-11-28 Fir de Conversatie Ike Devolder 
On Wed, Nov 28, 2012 at 05:35:03PM +0100, Bram Moolenaar wrote:
 
 Patch 7.3.737
 Problem:When using do_cmdline() recursively did_endif is not reset,
   causing messages to be overwritten.
 Solution:   Reset did_endif. (Christian Brabandt)
 Files:src/ex_docmd.c
 
 
 *** ../vim-7.3.736/src/ex_docmd.c 2012-10-03 18:24:55.0 +0200
 --- src/ex_docmd.c2012-11-28 17:25:27.0 +0100
 ***
 *** 1536,1541 
 --- 1536,1543 
   if_level = 0;
   #endif
   
 + did_endif = FALSE;  /* in case do_cmdline used recursively */
 + 
   --call_depth;
   return retval;
   }
 *** ../vim-7.3.736/src/version.c  2012-11-28 16:49:53.0 +0100
 --- src/version.c 2012-11-28 17:33:54.0 +0100
 ***
 *** 727,728 
 --- 727,730 
   {   /* Add new patch number below this line */
 + /**/
 + 737,
   /**/
 
 -- 
 From know your smileys:
  :-)  Funny
  |-)  Funny Oriental
  (-:  Funny Australian
 
  /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net   \\\
 ///sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
 \\\  an exciting new programming language -- http://www.Zimbu.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

this patch makes fails the tiny build.

i think the #endif must be below the 'did_endif = FALSE;'

attached small patch

-- 
Ike

-- 
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
diff -r eccae034465b src/ex_docmd.c
--- a/src/ex_docmd.cWed Nov 28 17:41:59 2012 +0100
+++ b/src/ex_docmd.cWed Nov 28 18:48:51 2012 +0100
@@ -1534,9 +1534,9 @@
  * :endif (could be :if x | foo | endif).
  */
 if_level = 0;
-#endif
 
 did_endif = FALSE;  /* in case do_cmdline used recursively */
+#endif
 
 --call_depth;
 return retval;

Re: Patch 7.3.730

2012-11-28 Fir de Conversatie Ike Devolder 
On Wed, Nov 28, 2012 at 05:13:12PM +0100, Christian Brabandt wrote:
 On Wed, November 28, 2012 15:25, Bram Moolenaar wrote:
 
  Patch 7.3.730
  Problem:Crash in PHP file when using syntastic. (Ike Devolder)
  Solution:   Avoid using NULL pointer. (Christian Brabandt)
  Files:  src/quickfix.c
 
 Huh, that was fast. I haven't gotten the time to check the valgrind
 question yet, that came up in that thread (I don't particular know
 valgrind very well) and additionally, we should remove the win_T
 argument to the call of qf_new_list() that was introduced with patch
 7.3.715.
 
 So here is that missing part.
 
 regards,
 Christian
 
 -- 
 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

thx for including this, it works all fine :)

-- 
Ike

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

Re: Patch 7.3.741

2012-11-28 Fir de Conversatie Ike Devolder 
On Wed, Nov 28, 2012 at 07:11:12PM +0100, Bram Moolenaar wrote:
 
 Patch 7.3.741 (after 7.3.737)
 Problem:Tiny build fails.
 Solution:   Move #ifdef. (Ike Devolder)
 Files:src/ex_docmd.c
 
 
 *** ../vim-7.3.740/src/ex_docmd.c 2012-11-28 17:34:43.0 +0100
 --- src/ex_docmd.c2012-11-28 19:09:39.0 +0100
 ***
 *** 1534,1542 
* :endif (could be :if x | foo | endif).
*/
   if_level = 0;
 - #endif
   
   did_endif = FALSE;  /* in case do_cmdline used recursively */
   
   --call_depth;
   return retval;
 --- 1534,1542 
* :endif (could be :if x | foo | endif).
*/
   if_level = 0;
   
   did_endif = FALSE;  /* in case do_cmdline used recursively */
 + #endif
   
   --call_depth;
   return retval;
 *** ../vim-7.3.740/src/version.c  2012-11-28 18:31:49.0 +0100
 --- src/version.c 2012-11-28 19:09:15.0 +0100
 ***
 *** 727,728 
 --- 727,730 
   {   /* Add new patch number below this line */
 + /**/
 + 741,
   /**/
 
 -- 
 From know your smileys:
  :-O-o   Smiley American tourist (note big mouth and camera)
 
  /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net   \\\
 ///sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
 \\\  an exciting new programming language -- http://www.Zimbu.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

I am truly sorry i have sent you an incomplete patch.
I should have double checked what i was sending.

The attached patch is the completion of the previous.

-- 
Ike

-- 
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
diff -r 6e1fe64cf440 src/ex_docmd.c
--- a/src/ex_docmd.cWed Nov 28 19:10:54 2012 +0100
+++ b/src/ex_docmd.cWed Nov 28 20:04:18 2012 +0100
@@ -1534,7 +1534,9 @@
  * :endif (could be :if x | foo | endif).
  */
 if_level = 0;
-
+#endif
+
+#ifdef FEAT_EVAL
 did_endif = FALSE;  /* in case do_cmdline used recursively */
 #endif

Re: Patch 7.3.715

2012-11-25 Fir de Conversatie Ike Devolder 
On Sun, Nov 25, 2012 at 02:29:59PM +0100, Christian Brabandt wrote:
 Hi everybody
 
 On Fr, 23 Nov 2012, Ike Devolder wrote:
  but the following i can reproduce every time:
  
  same configuration as before, same file (index.php)
  - open index.php (it still has the syntax error
  - :w (syntastic kicks in)
  - fix the error by adding ; after phpinfo()
  - :w (no more error in the file)
  - remove the ; again
  - :w (syntastic kicks in)
  - :w -E776: No location list
  
  index.php 2L, 16C written
  Error detected while processing function 
  SNR31_UpdateErrors..SNR31_AutoToggleLocList..SNR31_ShowLocList:
  line3:
  E776: No location list
  Press ENTER or type command to continue
  
  no crash in this case, if i continue after this issue i get a second 
  quickfix 
  buffer.
  
  thx for the intermediate fix it is definatly an improvement.
 
 The following patch seems to fix all the issues mentioned so far. It 
 basically does this by reverting Patch 7.3.715 and making sure, no data 
 is freed more than once. Unfortunately, I have not been able to come up 
 with a simple test case for the syntastic problem, that could be 
 included.
 
 If anybody can come up with such a test (using no plugin, but just using 
 vim -u NONE -U NONE -N) a tip is appreciated.
 
 regards,
 Christian
 -- 
 Ich bin geldgierig. Als Finanzminister muß man geldgierig sein.
   -- Hans Eichel
 
 -- 
 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

 diff --git a/src/quickfix.c b/src/quickfix.c
 --- a/src/quickfix.c
 +++ b/src/quickfix.c
 @@ -898,11 +898,7 @@
   * way with :grep'.
   */
  while (qi-qf_listcount  qi-qf_curlist + 1)
 -{
 - if (wp != NULL  wp-w_llist == qi)
 - wp-w_llist = NULL;
   qf_free(qi, --qi-qf_listcount);
 -}
  
  /*
   * When the stack is full, remove to oldest entry
 @@ -910,8 +906,6 @@
   */
  if (qi-qf_listcount == LISTCOUNT)
  {
 - if (wp != NULL  wp-w_llist == qi)
 - wp-w_llist = NULL;
   qf_free(qi, 0);
   for (i = 1; i  LISTCOUNT; ++i)
   qi-qf_lists[i - 1] = qi-qf_lists[i];
 @@ -2135,13 +2129,17 @@
  while (qi-qf_lists[idx].qf_count)
  {
   qfp = qi-qf_lists[idx].qf_start-qf_next;
 - vim_free(qi-qf_lists[idx].qf_start-qf_text);
 - vim_free(qi-qf_lists[idx].qf_start-qf_pattern);
 - vim_free(qi-qf_lists[idx].qf_start);
 + if (qi-qf_lists[idx].qf_title != NULL)
 + {
 + vim_free(qi-qf_lists[idx].qf_start-qf_text);
 + vim_free(qi-qf_lists[idx].qf_start-qf_pattern);
 + vim_free(qi-qf_lists[idx].qf_start);
 + }
   qi-qf_lists[idx].qf_start = qfp;
   --qi-qf_lists[idx].qf_count;
  }
 -vim_free(qi-qf_lists[idx].qf_title);
 +if (qi-qf_lists[idx].qf_title != NULL)
 + vim_free(qi-qf_lists[idx].qf_title);
  qi-qf_lists[idx].qf_title = NULL;
  }
  

thx all seems to be working fine with this patch

-- 
Ike

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

Re: Patch 7.3.715

2012-11-23 Fir de Conversatie Ike Devolder 
Op vrijdag 23 november 2012 21:47:47 schreef Christian Brabandt:
 Hi Bram!
 
 On Fr, 23 Nov 2012, Bram Moolenaar wrote:
  Christian Brabandt wrote:
   Hi Dominique!
   
   On Do, 22 Nov 2012, Dominique Pellé wrote:
I can't tell whether that's the right fix, but I confirm that
your proposed change avoids the crash at least.

wp-w_llist was set to NULL at line quickfix.c:914 which

was introduced by chane 3918 according to hg annotate:
 911 vimboss  644: if (qi-qf_listcount == LISTCOUNT)
 912 vimboss7: {
 913bram 3918:   if (wp != NULL  wp-w_llist == qi)
 914bram 3918:   wp-w_llist = NULL;
 915 vimboss  644:   qf_free(qi, 0);

===
changeset:   3918:4f0ddf4137ee
tag: v7-3-715
user:Bram Moolenaar b...@vim.org
date:Wed Nov 14 22:38:08 2012 +0100
files:   src/quickfix.c src/testdir/test49.ok
src/testdir/test49.vim src/version.c
description:
updated for version 7.3.715
Problem:Crash when calling setloclist() in BufUnload autocmd.
(Marcin

Szamotulski)

Solution:   Set w_llist to NULL when it was freed.  Also add a test.

(Christian Brabandt)

===

I find it odd that a function called qf_new_list() clears
wp-w_llist (set it to NULL) and does not set it back
to something else. The name of the function qf_new_list()
suggests that it should create another list, so perhaps
it should set wp-w_llist to something else. But I don't
understand the code here.
   
   Indeed. I think GET_LOC_LIST should be defined as
   ll_get_or_alloc_list(wp)
  
  ll_get_or_alloc_list() can still return NULL, thus your check is needed
  anyway.
 
 It is still not right so. I'll look into it later.
 
 regards,
 Christian

I have tested the patch on top of 725
had one segfault already (could not yet reproduce)

but the following i can reproduce every time:

same configuration as before, same file (index.php)
- open index.php (it still has the syntax error
- :w (syntastic kicks in)
- fix the error by adding ; after phpinfo()
- :w (no more error in the file)
- remove the ; again
- :w (syntastic kicks in)
- :w -E776: No location list

index.php 2L, 16C written
Error detected while processing function 
SNR31_UpdateErrors..SNR31_AutoToggleLocList..SNR31_ShowLocList:
line3:
E776: No location list
Press ENTER or type command to continue

no crash in this case, if i continue after this issue i get a second quickfix 
buffer.

thx for the intermediate fix it is definatly an improvement.

--Ike

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

Re: Patch 7.3.715

2012-11-22 Fir de Conversatie Ike Devolder 
On Thu, Nov 22, 2012 at 09:08:48AM +0100, Christian Brabandt wrote:
 On Thu, November 22, 2012 08:47, Ike Devolder wrote:
 [Patch 7.3.315]
  Anyone else having issues with this patch ?
 
  In my day to day work i use vim in combination with syntastic to do some
  linting for me. Since this patch i have frequent segfaults when
  syntastic kicks in. It seems related to this patch, maybe not immediatly
  due to setloclist itself but something related with quickfix.
 
  I'm sorry but atm i have no idea where to search.
 
  For now i'm stuck on 7.3.714 since that version can keep me working all
  day long.
 
  some output:
  --- start terminal output ---
  vim
  Vim: fataal signaal gevangen SEGV
 
  Vim: Finished.
  Segmentatiefout
  --- end terminal output ---
 
 It would help, if you can craft a reproducible example. I have no idea,
 what syntastic is doing.
 
 regards,
 Christian
 
 -- 
 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

When i'm back at home i'll try to find a way to reproduce this problem
consistently. I've had segfaults in many occasions but I dont yet have a
pattern to make sure the segfault happens.

I'm also not sure the problem lies in the patch itself but it was the
trigger.

-- 
Ike

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

Re: Patch 7.3.715

2012-11-22 Fir de Conversatie Ike Devolder 
Op donderdag 22 november 2012 20:24:40 schreef u:
 On Thu, Nov 22, 2012 at 03:08:14PM +0100, Ike Devolder wrote:
  On Thu, Nov 22, 2012 at 09:08:48AM +0100, Christian Brabandt wrote:
   On Thu, November 22, 2012 08:47, Ike Devolder wrote:
   [Patch 7.3.315]
   
Anyone else having issues with this patch ?

In my day to day work i use vim in combination with syntastic to do
some
linting for me. Since this patch i have frequent segfaults when
syntastic kicks in. It seems related to this patch, maybe not
immediatly
due to setloclist itself but something related with quickfix.

I'm sorry but atm i have no idea where to search.

For now i'm stuck on 7.3.714 since that version can keep me working
all
day long.

some output:
--- start terminal output ---
vim
Vim: fataal signaal gevangen SEGV

Vim: Finished.
Segmentatiefout
--- end terminal output ---
   
   It would help, if you can craft a reproducible example. I have no idea,
   what syntastic is doing.
   
   regards,
   Christian
  
  When i'm back at home i'll try to find a way to reproduce this problem
  consistently. I've had segfaults in many occasions but I dont yet have a
  pattern to make sure the segfault happens.
  
  I'm also not sure the problem lies in the patch itself but it was the
  trigger.
 
 i attach a minimum configuration + testfile where i have consistent
 segfaults on.
 
 within the archive you can find index.php, it has a syntax error, the
 following steps lead to segfault:
 
 vim index.php
 - :w (with the syntax error, syntastic kicks in)
 - goto end of phpinfo() and add ; :w (error is gone)
 - undo u
 - :w - segfault
 
 uname -a:
 Linux 3.6.7 #1 SMP PREEMPT Sun Nov 18 09:53:58 CET 2012 x86_64 GNU/Linux
 distribution Arch Linux
 
 configure:
 ./configure --prefix=/usr --localstatedir=/var/lib/vim \
   --mandir=/usr/share/man --with-compiledby=BlackEagle \
   --with-features=huge --enable-gpm --enable-acl --with-x=no \
   --disable-gui --enable-multibyte --enable-cscope \
   --disable-netbeans --enable-perlinterp=dynamic \
   --enable-pythoninterp=dynamic --enable-python3interp=dynamic \
   --enable-rubyinterp=dynamic --enable-luainterp=dynamic
 
 now tested with version 7.3.725, first noticed with 7.3.718 and then
 detected that the problem went away when going back to 7.3.714
 
 to be able to build ruby dynamic on my machine i also needed the patch
 of Ken Takata in the 7.3.721 thread:
 https://groups.google.com/forum/?fromgroups=#!topic/vim_dev/-pxkvVyWZdo
 
 thx for looking into this

ok i forgot on pre-requisite: you must have php installed so the syntastic 
plugin can run php -l to get the syntax error showing.

--Ike

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

Re: Patch 7.3.715

2012-11-21 Fir de Conversatie Ike Devolder 
On Wed, Nov 14, 2012 at 10:38:29PM +0100, Bram Moolenaar wrote:
 
 Patch 7.3.715
 Problem:Crash when calling setloclist() in BufUnload autocmd. (Marcin
   Szamotulski)
 Solution:   Set w_llist to NULL when it was freed.  Also add a test.
   (Christian Brabandt)
 Files:src/quickfix.c, src/testdir/test49.ok, 
 src/testdir/test49.vim
 
 
 *** ../vim-7.3.714/src/quickfix.c 2012-06-29 12:57:03.0 +0200
 --- src/quickfix.c2012-11-14 22:33:20.0 +0100
 ***
 *** 107,113 
   };
   
   static int  qf_init_ext __ARGS((qf_info_T *qi, char_u *efile, buf_T *buf, 
 typval_T *tv, char_u *errorformat, int newlist, linenr_T lnumfirst, linenr_T 
 lnumlast, char_u *qf_title));
 ! static void qf_new_list __ARGS((qf_info_T *qi, char_u *qf_title));
   static void ll_free_all __ARGS((qf_info_T **pqi));
   static int  qf_add_entry __ARGS((qf_info_T *qi, qfline_T **prevp, char_u 
 *dir, char_u *fname, int bufnum, char_u *mesg, long lnum, int col, int 
 vis_col, char_u *pattern, int nr, int type, int valid));
   static qf_info_T *ll_new_list __ARGS((void));
 --- 107,113 
   };
   
   static int  qf_init_ext __ARGS((qf_info_T *qi, char_u *efile, buf_T *buf, 
 typval_T *tv, char_u *errorformat, int newlist, linenr_T lnumfirst, linenr_T 
 lnumlast, char_u *qf_title));
 ! static void qf_new_list __ARGS((qf_info_T *qi, char_u *qf_title, win_T 
 *wp));
   static void ll_free_all __ARGS((qf_info_T **pqi));
   static int  qf_add_entry __ARGS((qf_info_T *qi, qfline_T **prevp, char_u 
 *dir, char_u *fname, int bufnum, char_u *mesg, long lnum, int col, int 
 vis_col, char_u *pattern, int nr, int type, int valid));
   static qf_info_T *ll_new_list __ARGS((void));
 ***
 *** 266,272 
   
   if (newlist || qi-qf_curlist == qi-qf_listcount)
   /* make place for a new list */
 ! qf_new_list(qi, qf_title);
   else if (qi-qf_lists[qi-qf_curlist].qf_count  0)
   /* Adding to existing list, find last entry. */
   for (qfprev = qi-qf_lists[qi-qf_curlist].qf_start;
 --- 266,272 
   
   if (newlist || qi-qf_curlist == qi-qf_listcount)
   /* make place for a new list */
 ! qf_new_list(qi, qf_title, curwin);
   else if (qi-qf_lists[qi-qf_curlist].qf_count  0)
   /* Adding to existing list, find last entry. */
   for (qfprev = qi-qf_lists[qi-qf_curlist].qf_start;
 ***
 *** 885,893 
* Prepare for adding a new quickfix list.
*/
   static void
 ! qf_new_list(qi, qf_title)
   qf_info_T   *qi;
   char_u  *qf_title;
   {
   int i;
   
 --- 885,894 
* Prepare for adding a new quickfix list.
*/
   static void
 ! qf_new_list(qi, qf_title, wp)
   qf_info_T   *qi;
   char_u  *qf_title;
 + win_T   *wp;
   {
   int i;
   
 ***
 *** 897,903 
 --- 898,908 
* way with :grep'.
*/
   while (qi-qf_listcount  qi-qf_curlist + 1)
 + {
 + if (wp != NULL  wp-w_llist == qi)
 + wp-w_llist = NULL;
   qf_free(qi, --qi-qf_listcount);
 + }
   
   /*
* When the stack is full, remove to oldest entry
 ***
 *** 905,910 
 --- 910,917 
*/
   if (qi-qf_listcount == LISTCOUNT)
   {
 + if (wp != NULL  wp-w_llist == qi)
 + wp-w_llist = NULL;
   qf_free(qi, 0);
   for (i = 1; i  LISTCOUNT; ++i)
   qi-qf_lists[i - 1] = qi-qf_lists[i];
 ***
 *** 3181,3187 
eap-cmdidx != CMD_vimgrepadd  eap-cmdidx != CMD_lvimgrepadd)
   || qi-qf_curlist == qi-qf_listcount)
   /* make place for a new list */
 ! qf_new_list(qi, *eap-cmdlinep);
   else if (qi-qf_lists[qi-qf_curlist].qf_count  0)
   /* Adding to existing list, find last entry. */
   for (prevp = qi-qf_lists[qi-qf_curlist].qf_start;
 --- 3188,3194 
eap-cmdidx != CMD_vimgrepadd  eap-cmdidx != CMD_lvimgrepadd)
   || qi-qf_curlist == qi-qf_listcount)
   /* make place for a new list */
 ! qf_new_list(qi, *eap-cmdlinep, curwin);
   else if (qi-qf_lists[qi-qf_curlist].qf_count  0)
   /* Adding to existing list, find last entry. */
   for (prevp = qi-qf_lists[qi-qf_curlist].qf_start;
 ***
 *** 3747,3753 
   
   if (action == ' ' || qi-qf_curlist == qi-qf_listcount)
   /* make place for a new list */
 ! qf_new_list(qi, title);
   else if (action == 'a'  qi-qf_lists[qi-qf_curlist].qf_count  0)
   /* Adding to existing list, find last entry. */
   for (prevp = qi-qf_lists[qi-qf_curlist].qf_start;
 --- 3754,3760 
   
   if (action == ' ' || qi-qf_curlist == qi-qf_listcount)
   /* make place for a new list */
 ! qf_new_list(qi, title, wp);
   else if (action == 'a'  qi-qf_lists[qi-qf_curlist].qf_count  0)
   /* Adding to existing list, find last entry. */
   for (prevp =

Re: Patch 7.3.721

2012-11-20 Fir de Conversatie Ike Devolder 
Op dinsdag 20 november 2012 16:59:30 schreef Bram Moolenaar:
 Patch 7.3.721
 Problem:Ruby interface defines local functions globally.
 Solution:   Make the functions static.
 Files:src/if_ruby.c
 
 
 *** ../vim-7.3.720/src/if_ruby.c  2012-10-03 18:06:55.0 +0200
 --- src/if_ruby.c 2012-11-20 15:35:46.0 +0100
 ***
 *** 336,346 
   #endif
 
   #ifdef RUBY19_OR_LATER
 ! SIGNED_VALUE rb_num2long_stub(VALUE x)
   {
   return dll_rb_num2long(x);
   }
 ! VALUE rb_int2big_stub(SIGNED_VALUE x)
   {
   return dll_rb_int2big(x);
   }
 --- 336,346 
   #endif
 
   #ifdef RUBY19_OR_LATER
 ! static SIGNED_VALUE rb_num2long_stub(VALUE x)
   {
   return dll_rb_num2long(x);
   }
 ! static VALUE rb_int2big_stub(SIGNED_VALUE x)
   {
   return dll_rb_int2big(x);
   }
 *** ../vim-7.3.720/src/version.c  2012-11-20 16:56:49.0 +0100
 --- src/version.c 2012-11-20 16:58:32.0 +0100
 ***
 *** 727,728 
 --- 727,730 
   {   /* Add new patch number below this line */
 + /**/
 + 721,
   /**/

build failure here:
if_ruby.c:339:21: error: static declaration of 'rb_num2long_stub' follows non-
static declaration

./configure --prefix=/usr --localstatedir=/var/lib/vim \
--mandir=/usr/share/man \
--with-features=huge --enable-gpm --enable-acl --with-x=no \
--disable-gui --enable-multibyte --enable-cscope \
--disable-netbeans --enable-perlinterp=dynamic \
--enable-pythoninterp=dynamic --enable-python3interp=dynamic \
--enable-rubyinterp=dynamic --enable-luainterp=dynamic
make

--Ike

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

Re: Patch 7.3.697

2012-10-21 Fir de Conversatie Ike Devolder 
On Sunday, October 21, 2012 01:40:45 AM Bram Moolenaar wrote:
 Patch 7.3.697
 Problem:Leaking resources when setting GUI font.
 Solution:   Free the font. (Ken Takata)
 Files:  src/syntax.c
 
 
 *** ../vim-7.3.696/src/syntax.c   2012-07-19 17:39:01.0 +0200
 --- src/syntax.c  2012-10-21 01:37:19.0 +0200
 ***
 *** 105,111 
   # ifdef FEAT_XFONTSET
   static GuiFontset fontset_name2handle __ARGS((char_u *name, int
 fixed_width)); # endif
 ! static void hl_do_font __ARGS((int idx, char_u *arg, int do_normal, int
 do_menu, int do_tooltip)); #endif
 
   /*
 --- 105,111 
   # ifdef FEAT_XFONTSET
   static GuiFontset fontset_name2handle __ARGS((char_u *name, int
 fixed_width)); # endif
 ! static void hl_do_font __ARGS((int idx, char_u *arg, int do_normal, int
 do_menu, int do_tooltip, int free_font)); #endif
 
   /*
 ***
 *** 7259,7272 
   HL_TABLE()[idx].sg_fontset = NOFONTSET;
   # endif
   hl_do_font(idx, arg, is_normal_group, is_menu_group,
 ! is_tooltip_group);
 
   # ifdef FEAT_XFONTSET
   if (HL_TABLE()[idx].sg_fontset != NOFONTSET)
   {
 ! /* New fontset was accepted. Free the old one, if there was
 !  * one.
 !  */
   gui_mch_free_fontset(temp_sg_fontset);
   vim_free(HL_TABLE()[idx].sg_font_name);
   HL_TABLE()[idx].sg_font_name = vim_strsave(arg);
 --- 7259,7271 
   HL_TABLE()[idx].sg_fontset = NOFONTSET;
   # endif
   hl_do_font(idx, arg, is_normal_group, is_menu_group,
 !  is_tooltip_group, FALSE);
 
   # ifdef FEAT_XFONTSET
   if (HL_TABLE()[idx].sg_fontset != NOFONTSET)
   {
 ! /* New fontset was accepted. Free the old one, if there
 !  * was one. */
   gui_mch_free_fontset(temp_sg_fontset);
   vim_free(HL_TABLE()[idx].sg_font_name);
   HL_TABLE()[idx].sg_font_name = vim_strsave(arg);
 ***
 *** 7277,7284 
   if (HL_TABLE()[idx].sg_font != NOFONT)
   {
   /* New font was accepted. Free the old one, if there was
 !  * one.
 !  */
   gui_mch_free_font(temp_sg_font);
   vim_free(HL_TABLE()[idx].sg_font_name);
   HL_TABLE()[idx].sg_font_name = vim_strsave(arg);
 --- 7276,7282 
   if (HL_TABLE()[idx].sg_font != NOFONT)
   {
   /* New font was accepted. Free the old one, if there was
 !  * one. */
   gui_mch_free_font(temp_sg_font);
   vim_free(HL_TABLE()[idx].sg_font_name);
   HL_TABLE()[idx].sg_font_name = vim_strsave(arg);
 ***
 *** 8064,8075 
* Get the font or fontset for one highlight group.
*/
   static void
 ! hl_do_font(idx, arg, do_normal, do_menu, do_tooltip)
   int idx;
   char_u  *arg;
   int do_normal;  /* set normal font */
   int do_menu UNUSED; /* set menu font */
   int do_tooltip UNUSED;  /* set tooltip font */
   {
   # ifdef FEAT_XFONTSET
   /* If 'guifontset' is not empty, first try using the name as a
 --- 8062,8074 
* Get the font or fontset for one highlight group.
*/
   static void
 ! hl_do_font(idx, arg, do_normal, do_menu, do_tooltip, free_font)
   int idx;
   char_u  *arg;
   int do_normal;  /* set normal font */
   int do_menu UNUSED; /* set menu font */
   int do_tooltip UNUSED;  /* set tooltip font */
 + int free_font;  /* free current font/fontset */
   {
   # ifdef FEAT_XFONTSET
   /* If 'guifontset' is not empty, first try using the name as a
 ***
 *** 8083,8088 
 --- 8082,8089 
 
   || do_tooltip
 
   #  endif
   )
 + if (free_fontset)
 + gui_mch_free_fontset(HL_TABLE()[idx].sg_fontset);
   HL_TABLE()[idx].sg_fontset = fontset_name2handle(arg, 0
   #  ifdef FONTSET_ALWAYS
 
   || do_menu
 
 ***
 *** 8093,8100 
   );
   if (HL_TABLE()[idx].sg_fontset != NOFONTSET)
   {
 ! /* If it worked and it's the Normal group, use it as the
 !  * normal fontset.  Same for the Menu group. */
   if (do_normal)
   gui_init_font(arg, TRUE);
   #   if (defined(FEAT_GUI_MOTIF) || defined(FEAT_GUI_ATHENA)) 
 defined(FEAT_MENU) --- 8094,8101 
   );
   if (HL_TABLE()[idx].sg_fontset != NOFONTSET)
   {
 ! /* If it worked and it's the Normal group, use it as the normal
 !  * fontset.  Same for the Menu group. */

Re: Patch 7.3.697

2012-10-21 Fir de Conversatie Ike Devolder 
On Sunday, October 21, 2012 09:25:54 PM Bram Moolenaar wrote:
 Ike Devolder wrote:
  On Sunday, October 21, 2012 01:40:45 AM Bram Moolenaar wrote:
   Patch 7.3.697
   Problem:Leaking resources when setting GUI font.
   Solution:   Free the font. (Ken Takata)
   Files:  src/syntax.c
   
   
   *** ../vim-7.3.696/src/syntax.c   2012-07-19 17:39:01.0 +0200
   --- src/syntax.c  2012-10-21 01:37:19.0 +0200
 
 [...]
 
  from this patch on i get the following error:
  syntax.c: In function 'hl_do_font':
  syntax.c:8085:6: error: 'free_fontset' undeclared (first use in this
  function) syntax.c:8085:6: note: each undeclared identifier is reported
  only once for each function it appears in
  make[1]: *** [objects/syntax.o] Error 1
  
  the attached patch fixes the problem
 
 Sorry about that.  I don't build for all different GUIs, takes too much
 time.

no biggy, thanks for picking it up so quickly and for this great editor i use 
every day :)

--Ike

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

Re: Patch 7.3.662

2012-09-24 Fir de Conversatie Ike Devolder 
On Sun, Sep 23, 2012 at 11:58:21PM -0700, mattn wrote:
 On Sunday, September 23, 2012 6:58:36 PM UTC+9, Bram Moolenaar wrote:
  Ken Takata wrote:
  
  
  
   Hi Bram and mattn,
  
   
  
   Please try the attached patch.
  
   It seems that rb_num2uint and some other functions are exported
  
   on 64-bit platforms but not on 32-bit platforms.
  
  
  
  Thanks.  Can a few people try this out?
 
 I've make sure this patch working on my windows+ruby193.
 
 -- 
 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

I can also confirm the additional patch makes it work all fine on i686
and x86_64 archlinux and ruby 1.9.3p194

-- 
Ike

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

Re: Patch 7.3.597

2012-07-10 Fir de Conversatie Ike Devolder 
Op dinsdag 10 juli 2012 16:49:38 schreef Bram Moolenaar:
 Patch 7.3.597
 Problem:'clipboard' autoselect only applies to the * register. (Sergey
 Vakulenko)
 Solution:   Make 'autoselect' work for the + register. (Christian Brabant)
   Add the autoselectplus option in 'clipboard' and the P flag in
   'guioptions'.
 Files:runtime/doc/options.txt, src/normal.c, src/ops.c, 
 src/screen.c,
   src/ui.c, src/globals.h, src/proto/ui.pro, src/option.h, src/gui.c
 
 
 *** ../vim-7.3.596/runtime/doc/options.txt2012-02-20 22:18:23.0
 +0100 --- runtime/doc/options.txt 2012-07-10 15:40:35.0 +0200
 ***
 *** 1437,1442 
 --- 1452,1458 
   This option is a list of comma separated names.
   These names are recognized:
 
 + *clipboard-unnamed*
   unnamed When included, Vim will use the clipboard register '*'
   for all yank, delete, change and put operations which
   would normally go to the unnamed register.  When a
 ***
 *** 1446,1460 
   explicitly accessed using the * notation.  Also see
 
   |gui-clipboard|.
 
   unnamedplus A variant of unnamed flag which uses the clipboard
   register '+' (|quoteplus|) instead of register '*' for
   all operations except yank.  Yank shall copy the text
   into register '+' and also into '*' when unnamed is
   included.
 ! Only available with the |+x11| feature.
   Availability can be checked with: 
   if has('unnamedplus')
   
   autoselect  Works like the 'a' flag in 'guioptions': If present,
   then whenever Visual mode is started, or the Visual
   area extended, Vim tries to become the owner of the
 --- 1462,1478 
   explicitly accessed using the * notation.  Also see
 
   |gui-clipboard|.
 
 + *clipboard-unnamedplus*
   unnamedplus A variant of unnamed flag which uses the clipboard
   register '+' (|quoteplus|) instead of register '*' for
   all operations except yank.  Yank shall copy the text
   into register '+' and also into '*' when unnamed is
   included.
 ! Only available with the |+X11| feature.
   Availability can be checked with: 
   if has('unnamedplus')
   
 + *clipboard-autoselect*
   autoselect  Works like the 'a' flag in 'guioptions': If present,
   then whenever Visual mode is started, or the Visual
   area extended, Vim tries to become the owner of the
 ***
 *** 1466,1474 
 --- 1484,1499 
   autoselect flag is used.
   Also applies to the modeless selection.
 
 + *clipboard-autoselectplus*
 + autoselectplus  Like autoselect but using the + register instead of
 + the * register.  Compare to the 'P' flag in
 + 'guioptions'.
 +
 + *clipboard-autoselectml*
   autoselectmlLike autoselect, but for the modeless selection
   only.  Compare to the 'A' flag in 'guioptions'.
 
 + *clipboard-html*
   htmlWhen the clipboard contains HTML, use this when
   pasting.  When putting text on the clipboard, mark it
   as HTML.  This works to copy rendered HTML from
 ***
 *** 1479,1484 
 --- 1504,1510 
   Only supported for GTK version 2 and later.
   Only available with the |+multi_byte| feature.
 
 + *clipboard-exclude*
   exclude:{pattern}
   Defines a pattern that is matched against the name of
   the terminal 'term'.  If there is a match, no
 ***
 *** 3547,3552 
 --- 3600,3608 
   windowing system's global selection unless explicitly told to
   by a yank or delete operation for the * register.
   The same applies to the modeless selection.
 + *'go-P'*
 +   'P'   Like autoselect but using the + register instead of the *
 + register.
   *'go-A'*
 'A'   Autoselect for the modeless selection.  Like 'a', but only
   applies to the modeless

Re: Patch 7.3.599

2012-07-10 Fir de Conversatie Ike Devolder 
Op dinsdag 10 juli 2012 18:32:08 schreef Bram Moolenaar:
 Patch 7.3.599 (after 7.3.597)
 Problem:Missing change in one file.
 Solution:   Patch for changed clip_autoselect().
 Files:  src/option.c
 
 
 *** ../vim-7.3.598/src/option.c   2012-06-29 15:51:26.0 +0200
 --- src/option.c  2012-07-10 16:35:07.0 +0200
 ***
 *** 7377,7383 
   check_clipboard_option()
   {
   int new_unnamed = 0;
 ! int new_autoselect = FALSE;
   int new_autoselectml = FALSE;
   int new_html = FALSE;
   regprog_T   *new_exclude_prog = NULL;
 --- 7377,7384 
   check_clipboard_option()
   {
   int new_unnamed = 0;
 ! int new_autoselect_star = FALSE;
 ! int new_autoselect_plus = FALSE;
   int new_autoselectml = FALSE;
   int new_html = FALSE;
   regprog_T   *new_exclude_prog = NULL;
 ***
 *** 7398,7410 
   p += 11;
   }
   else if (STRNCMP(p, autoselect, 10) == 0
 !  (p[10] == ',' || p[10] == NUL))
   {
 ! new_autoselect = TRUE;
   p += 10;
   }
   else if (STRNCMP(p, autoselectml, 12) == 0
 !  (p[12] == ',' || p[12] == NUL))
   {
   new_autoselectml = TRUE;
   p += 12;
 --- 7399,7417 
   p += 11;
   }
   else if (STRNCMP(p, autoselect, 10) == 0
 !  (p[10] == ',' || p[10] == NUL))
   {
 ! new_autoselect_star = TRUE;
   p += 10;
   }
 + else if (STRNCMP(p, autoselectplus, 14) == 0
 +  (p[14] == ',' || p[14] == NUL))
 + {
 + new_autoselect_plus = TRUE;
 + p += 14;
 + }
   else if (STRNCMP(p, autoselectml, 12) == 0
 !  (p[12] == ',' || p[12] == NUL))
   {
   new_autoselectml = TRUE;
   p += 12;
 ***
 *** 7433,7439 
   if (errmsg == NULL)
   {
   clip_unnamed = new_unnamed;
 ! clip_autoselect = new_autoselect;
   clip_autoselectml = new_autoselectml;
   clip_html = new_html;
   vim_free(clip_exclude_prog);
 --- 7440,7447 
   if (errmsg == NULL)
   {
   clip_unnamed = new_unnamed;
 ! clip_autoselect_star = new_autoselect_star;
 ! clip_autoselect_plus = new_autoselect_plus;
   clip_autoselectml = new_autoselectml;
   clip_html = new_html;
   vim_free(clip_exclude_prog);
 *** ../vim-7.3.598/src/version.c  2012-07-10 17:14:50.0 +0200
 --- src/version.c 2012-07-10 18:30:17.0 +0200
 ***
 *** 716,717 
 --- 716,719 
   {   /* Add new patch number below this line */
 + /**/
 + 599,
   /**/

thx, all builds fine now

--Ike

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

Re: Patch 7.3.590

2012-07-06 Fir de Conversatie Ike Devolder 
Op vrijdag 6 juli 2012 09:12:52 schreef Hisashi T Fujinaka:
 This isn't showing up in mercurial (or whatever the repo is, I have it
 wrapped in a script and, if you remember, I alwasy forget what version
 control Bram uses.)
 
 On Fri, 6 Jul 2012, Bram Moolenaar wrote:
  Patch 7.3.590
  Problem:The ' and ' marks cannot be set directly.
  Solution:   Allow setting ' and '. (Christian Brabandt)
  Files:  src/mark.c
  
  
  *** ../vim-7.3.589/src/mark.c   2011-02-25 15:11:17.0 +0100
  --- src/mark.c  2012-07-06 17:47:23.0 +0200
  ***
  *** 98,103 
  --- 98,116 
  
  return OK;
  
   }
  
  + #ifdef FEAT_VISUAL
  + if (c == '')
  + {
  +   curbuf-b_visual.vi_start = *pos;
  +   return OK;
  + }
  + if (c == '')
  + {
  +   curbuf-b_visual.vi_end = *pos;
  +   return OK;
  + }
  + #endif
  +
  
   #ifndef EBCDIC
   
   if (c  'z')   /* some islower() and isupper() cannot handle
   
  characters above 127 */
  
  *** ../vim-7.3.589/src/version.c2012-07-06 16:49:37.0 +0200
  --- src/version.c   2012-07-06 17:49:00.0 +0200
  ***
  *** 716,717 
  --- 716,719 
  
   {   /* Add new patch number below this line */
  
  + /**/
  + 590,
  
   /**/

Yep it is not showing up yet, but it will come eventually :p

--Ike

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

Re: Compiling Vim with +ruby support (vim 7.3 - ruby 1.9.3) on Amd64

2012-06-20 Fir de Conversatie Ike Devolder 
Op dinsdag 19 juni 2012 20:32:26 schreef Lomig Enfroy:
 Le mercredi 20 juin 2012 04:14:39 UTC+2, Thomas Dziedzic a écrit :
  On Tue, Jun 19, 2012 at 6:51 PM, Lomig wrote:
   Hello list :)
   
   As I have not been able to find a compiled package of vim with a +ruby
   support for my distribution (Archlinux), I have tried for 2 days to
   compile it myself from the mercurial repository... Though without any
   success for now.
   
   After a configure that was meant to be as close as possible to the one
   used by my previous binary package,
  
  There might be some confusion with the naming in archlinux.
  vim the package contains a minimally configured vim.
  gvim the package contains vim + gvim with ruby + python + lots of
  other options enabled.
  All you need for ruby support is to do pacman -S gvim and you should be
  done.
  
  
  ~  vim --version | grep +ruby
  
   +python -python3 +quickfix +reltime +rightleft +ruby +scrollbind +signs
  
  ~  uname -a
  Linux archlinux 3.4.3-1-ARCH #1 SMP PREEMPT Mon Jun 18 08:28:29 CEST
  2012 x86_64 GNU/Linux
  ~  pacman -Q gvim
  gvim 7.3.547-1
  
  Cheers
 
 My answer to your post has disappeared :/
 
 So, as I was saying, I use the Herecura version of this package,  because it
 provides a gvim-Qt flavor, and I try to use only one framework on a machine
 - and I do develop in Qt and QML myself.
 
 I tried to compile (Herecura's) gvim with gtk2 support instead to use a
 patch-free version, and be sure that it was not the source of my problem --
 And it was not :)
 
 I would be really glad to use your official package if you would consider
 the integration of a Qt version, though :)
 
 Regards,
 Lomig
 
 --
 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

Hi,

the reason ruby 1.9.3 is not working with 'my' herecura flavor is because of 
the dynamic loading. i wanted to provide a vim/gvim with as much as possible 
functionallity, with as less as possible hard dependencies. if you build the 
packages as they are built in archlinux, doing so having hard dependencies on 
ruby, python, perl, lua, ... then it will work just fine (--enable-
rubyinterp=yes)

i have posted in this mailinglist before for a solution for the dynamic ruby 
stuff but found no answer yet.

http://code.google.com/p/vim/issues/detail?id=49

--Ike

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