Re: [PATCH] (15/?) More possible memory leak fixes and error checking

2013-05-26 Fir de Conversatie ZyX
# HG changeset patch # User ZyX kp-...@ya.ru # Date 1369520053 -14400 # Node ID 484d93ca3c0244e847c101b6f550f8b8c94ae6f6 # Parent 72a773f3fc682f63c79d1b8481626fdec3198ddd More possible memory leak fixes and error checking diff -r 72a773f3fc68 -r 484d93ca3c02 src/eval.c --- a/src/eval.c

Re: [PATCH] (16/?) Python built-in compatibilty: make OutputWritelines support any sequence object

2013-05-26 Fir de Conversatie ZyX
# HG changeset patch # User ZyX kp-...@ya.ru # Date 1369521415 -14400 # Node ID 0b272b1d066da1229ca6dd2519a3456b4a073f31 # Parent 484d93ca3c0244e847c101b6f550f8b8c94ae6f6 Make OutputWritelines support any sequence objects diff -r 484d93ca3c02 -r 0b272b1d066d src/if_py_both.h ---

Re: [PATCH] (17/?) Core: improve f_function

2013-05-26 Fir de Conversatie ZyX
# HG changeset patch # User ZyX kp-...@ya.ru # Date 1369550131 -14400 # Node ID 542435007988a8d87b3739506422ebc7d58f3910 # Parent 0b272b1d066da1229ca6dd2519a3456b4a073f31 Change f_function to use new get_expanded_name function Reason: fixes the following bugs: function() call succeeds, but

Re: [PATCH] (19/?) Add FunctionRepr

2013-05-26 Fir de Conversatie ZyX
# HG changeset patch # User ZyX kp-...@ya.ru # Date 1369552448 -14400 # Node ID 3fb04fcf34ebf32f217f724f85064879e29b7fe4 # Parent abd30ecabb334396aabab4b284b008d57cbedd16 Add FunctionRepr diff -r abd30ecabb33 -r 3fb04fcf34eb src/if_py_both.h --- a/src/if_py_both.h Sun May 26 11:11:34 2013 +0400

Re: [PATCH] (7.5/?) Fix SEGV with unnamed buffer

2013-05-26 Fir de Conversatie ZyX
It would be better to merge this and (7/?) into one patch. # HG changeset patch # User ZyX kp-...@ya.ru # Date 1369556544 -14400 # Node ID dacaaf15b8c8f00d0d2bb0c9fb4972608844cc4d # Parent 3fb04fcf34ebf32f217f724f85064879e29b7fe4 Fix SEGV diff -r 3fb04fcf34eb -r dacaaf15b8c8 src/if_py_both.h

Re: Some performance figures for the new regexp engine

2013-05-26 Fir de Conversatie Bram Moolenaar
Marc Weber wrote: If we are at it - and other tools are already faster, is there a chance to also add a engine 3 reusing the fast code? If we partially break with VimL (by improving python support) why not also add a commonly used regex syntax if speed is different. I've tried

Re: bug with 7.3.981

2013-05-26 Fir de Conversatie Bram Moolenaar
Cesar Romani wrote: On 21/05/2013 05:04 a.m., Bram Moolenaar wrote: Cesar Romani wrote: By updating from 7.3.969 to 7.3.981 the following command doesn't work anymore: %s//\=''.matchstr(getline(.), '\d\+\ze').'' Let's say I use this command on the following text:

Re: Patch 7.3.1019

2013-05-26 Fir de Conversatie John Little
This patch gives me a compile warning (from gcc -Wshadow) regexp_nfa.c: In function ‘nfa_regatom’: regexp_nfa.c:830:8: warning: declaration of ‘i’ shadows a previous local [-Wshadow] regexp_nfa.c:607:10: warning: shadowed declaration is here [-Wshadow] The declaration of int i; at the beginning

Re: [PATCH] (21/?) Make vim.List more like built-in list

2013-05-26 Fir de Conversatie ZyX
There is still a bunch of not supported methods present in python list(). # HG changeset patch # User ZyX kp-...@ya.ru # Date 1369570433 -14400 # Node ID f7c3343a64a716b37dd0f8a3165d16ab93483c24 # Parent 570cdbf800e8790e250e4e5beadac7b489960c8e Make vim.List more like built-in list: - Make it

Re: [PATCH] (22/?) Make vim.Function subclassable, add constructor

2013-05-26 Fir de Conversatie ZyX
# HG changeset patch # User ZyX kp-...@ya.ru # Date 1369558368 -14400 # Node ID 1ea9c3409a13d2cf25faffc8ae0b832a112dad0c # Parent f7c3343a64a716b37dd0f8a3165d16ab93483c24 Make vim.Function subclassable diff -r f7c3343a64a7 -r 1ea9c3409a13 src/eval.c --- a/src/eval.cSun May 26 16:13:53

Re: [PATCH] (23/?) Check for errors in a number of places, change some exception messages

2013-05-26 Fir de Conversatie ZyX
# HG changeset patch # User ZyX kp-...@ya.ru # Date 1369564686 -14400 # Node ID 44b8f018c4fc76aaeedd9424fc987ce31593fafb # Parent 1ea9c3409a13d2cf25faffc8ae0b832a112dad0c Check for errors in a number of places, change some exception messages diff -r 1ea9c3409a13 -r 44b8f018c4fc src/if_py_both.h

Re: [PATCH] (24/?) Use more standard PyMapping_Keys

2013-05-26 Fir de Conversatie ZyX
Similar approach is used in python’s dict.update. # HG changeset patch # User ZyX kp-...@ya.ru # Date 1369566559 -14400 # Node ID b5a15b33b9d26a9bb23088c31935616862d604c0 # Parent 44b8f018c4fc76aaeedd9424fc987ce31593fafb Made pymap_to_tv use more standard PyMapping_Keys diff -r 44b8f018c4fc -r

Patch 7.3.1021

2013-05-26 Fir de Conversatie Bram Moolenaar
Patch 7.3.1021 Problem:New regexp engine does not ignore order of composing chars. Solution: Ignore composing chars order. Files: src/regexp_nfa.c, src/testdir/test95.in, src/testdir/test95.ok *** ../vim-7.3.1020/src/regexp_nfa.c2013-05-25 22:04:19.0 +0200 ---

Re: Error using A with wrap on

2013-05-26 Fir de Conversatie Bram Moolenaar
Axel Bender wrote: When wrap is on and the line ends just before the buffer's right margin (i.e. right before the scrollbar - visible), and A is used to append a character to the end of the line, the cursor is positioned in INSERT mode after the 5th column of the next line. This does not

Re: Patch 7.3.1019

2013-05-26 Fir de Conversatie Bram Moolenaar
John Little wrote: This patch gives me a compile warning (from gcc -Wshadow) regexp_nfa.c: In function ‘nfa_regatom’: regexp_nfa.c:830:8: warning: declaration of ‘i’ shadows a previous local [-Wshadow] regexp_nfa.c:607:10: warning: shadowed declaration is here [-Wshadow] The

Re: [PATCH] (26/?) Update documentation

2013-05-26 Fir de Conversatie ZyX
# HG changeset patch # User ZyX kp-...@ya.ru # Date 1369572430 -14400 # Node ID a9cd5f321ccca75a8339146091ba461fae168b34 # Parent cfff09398b373d2dbc3fa24fe0d2b20f4ce83d5d Update documentation diff -r cfff09398b37 -r a9cd5f321ccc runtime/doc/if_pyth.txt --- a/runtime/doc/if_pyth.txt Sun May 26

Re: [PATCH] (27/?) Don’t load autoload functions on vim.Function creation

2013-05-26 Fir de Conversatie ZyX
# HG changeset patch # User ZyX kp-...@ya.ru # Date 1369572460 -14400 # Node ID 31b317a2cb89e8ad98bfa56ecbe2a668a7fd3a9a # Parent a9cd5f321ccca75a8339146091ba461fae168b34 Don’t load autoload functions on vim.Function creation diff -r a9cd5f321ccc -r 31b317a2cb89 src/eval.c --- a/src/eval.c

Patch 7.3.1022

2013-05-26 Fir de Conversatie Bram Moolenaar
Patch 7.3.1022 Problem:Compiler warning for shadowed variable. (John Little) Solution: Move declaration, rename variables. Files: src/regexp_nfa.c *** ../vim-7.3.1021/src/regexp_nfa.c2013-05-26 14:32:01.0 +0200 --- src/regexp_nfa.c2013-05-26 14:50:27.0 +0200

Patch 7.3.1023

2013-05-26 Fir de Conversatie Bram Moolenaar
Patch 7.3.1023 Problem:Searching for composing char only and using \Z has different results. Solution: Make it match the composing char, matching everything is not useful. Files: src/regexp_nfa.c, src/testdir/test95.in, src/testdir/test95.ok ***

Re: Error using A with wrap on

2013-05-26 Fir de Conversatie Axel Bender
Sorry, forgot to start with -i NONE -N -u NONE --noplugin -U NONE when I provided the sample. Using the above arguments, set columns=54 number wrap showbreak=(cnt) can be used to reproduce the problem (which also presents itself when using a with the cursor being in the last column) with the

Re: [PATCH] (20.5/?) Make vim.Dictionary mimic built-in one

2013-05-26 Fir de Conversatie ZyX
# HG changeset patch # User ZyX kp-...@ya.ru # Date 1369579264 -14400 # Node ID a4ad9503285f0829d69784faf0c1b8fb06dc2c0f # Parent 31b317a2cb89e8ad98bfa56ecbe2a668a7fd3a9a Fix compilation in python-2.3 diff -r 31b317a2cb89 -r a4ad9503285f src/if_python.c --- a/src/if_python.c Sun May 26

Re: Feature request: vimdiff and foldmethod

2013-05-26 Fir de Conversatie Christian Brabandt
Hi Gary! On Sa, 25 Mai 2013, Gary Johnson wrote: On 2013-05-25, Marcin Szamotulski wrote: Dear Vim Dev, I have a proposition for a small but nice feature. Whenever I do diff then diffo (or diffo!) some settings are lost, for example the foldmethod is always reset to manual. It

Re: [PATCH] (0/?) Another series of Python patches

2013-05-26 Fir de Conversatie ZyX
This is the last huge hunk I can write before the end of May. Other parts of RFC are to be delayed. -- -- 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 ---

Patch 7.3.1024

2013-05-26 Fir de Conversatie Bram Moolenaar
Patch 7.3.1024 Problem:New regexp: End of matching pattern not set correctly. (Cesar Romani) Solution: Quit the loop after finding the match. Store nfa_has_zend in the program. Files: src/regexp_nfa.c, src/testdir/test64.in, src/testdir/test64.ok,

Patch 7.3.1025

2013-05-26 Fir de Conversatie Bram Moolenaar
Patch 7.3.1025 Problem:New regexp: not matching newline in string. (Marc Weber) Solution: Check for \n character. Files: src/regexp_nfa.c, src/testdir/test64.in, src/testdir/test64.ok *** ../vim-7.3.1024/src/regexp_nfa.c2013-05-26 16:57:23.0 +0200 --- src/regexp_nfa.c

[patch] removed useless 'if' in regexp_nfa.c

2013-05-26 Fir de Conversatie Dominique Pellé
Attached is a nitpicky patch to remove a useless 'if' in regexp_nfa.c Regards Dominique -- -- 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

Patch 7.3.1026

2013-05-26 Fir de Conversatie Bram Moolenaar
Patch 7.3.1026 Problem:New regexp: pattern that includs a new-line matches too early. (john McGowan) Solution: Do not start searching in the second line. Files: src/regexp_nfa.c, src/testdir/test64.in, src/testdir/test64.ok *** ../vim-7.3.1025/src/regexp_nfa.c

Re: Error using A with wrap on

2013-05-26 Fir de Conversatie Bram Moolenaar
Axel Bender wrote: Sorry, forgot to start with -i NONE -N -u NONE --noplugin -U NONE when I provided the sample. Using the above arguments, set columns=54 number wrap showbreak=(cnt) can be used to reproduce the problem (which also presents itself when using a with the cursor being in the

Re: [PATCH] (0/?) Another series of Python patches

2013-05-26 Fir de Conversatie Bram Moolenaar
ZyX wrote: This is the last huge hunk I can write before the end of May. Other parts of RFC are to be delayed. Thanks for all your work! I have a few more patches for Python listed, hopefully I can include them all this week. -- Engineers understand that their appearance only bothers other

Patch 7.3.1027

2013-05-26 Fir de Conversatie Bram Moolenaar
Patch 7.3.1027 Problem:New regexp performance: Calling no_Magic() very often. Solution: Remove magicness inline. Files: src/regexp_nfa.c *** ../vim-7.3.1026/src/regexp_nfa.c2013-05-26 18:40:11.0 +0200 --- src/regexp_nfa.c2013-05-26 19:18:32.0 +0200

Re: Error using A with wrap on

2013-05-26 Fir de Conversatie Axel Bender
Hm, where was your cursor after A? Did you get a new line before entering a character? Here the cursor would be positioned in the second line (the one with the numbers), at column 5 in INSERT mode. Only after I enter a character I would get a new line... -- -- You received this message from

Re: [patch] removed useless 'if' in regexp_nfa.c

2013-05-26 Fir de Conversatie Bram Moolenaar
Dominique Pelle wrote: Attached is a nitpicky patch to remove a useless 'if' in regexp_nfa.c Thanks, I'll include it with one of the patches. -- Normal people believe that if it ain't broke, don't fix it. Engineers believe that if it ain't broke, it doesn't have enough features yet.

Re: Error using A with wrap on

2013-05-26 Fir de Conversatie Bram Moolenaar
Axel Bender wrote: Hm, where was your cursor after A? Did you get a new line before entering a character? Here the cursor would be positioned in the second line (the one with the numbers), at column 5 in INSERT mode. Only after I enter a character I would get a new line... The cursor is

Re: Error using A with wrap on

2013-05-26 Fir de Conversatie Axel Bender
Ok, I take that, but what about the incorrect offset in the second line? -- -- 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

Re: Feature request: vimdiff and foldmethod

2013-05-26 Fir de Conversatie Christian Brabandt
Hi Marcin! On Sa, 25 Mai 2013, Marcin Szamotulski wrote: I have a proposition for a small but nice feature. Whenever I do diff then diffo (or diffo!) some settings are lost, for example the foldmethod is always reset to manual. It would be nice if vim could restore fdm setting to what it

Patch 7.3.1028

2013-05-26 Fir de Conversatie Bram Moolenaar
Patch 7.3.1028 Problem:New regexp performance: Copying a lot of position state. Solution: Only copy the sub-expressions that are being used. Files: src/regexp_nfa.c, src/regexp.h *** ../vim-7.3.1027/src/regexp_nfa.c2013-05-26 19:19:48.0 +0200 --- src/regexp_nfa.c

Re: Feature request: vimdiff and foldmethod

2013-05-26 Fir de Conversatie Marcin Szamotulski
On 21:00 Sun 26 May , Christian Brabandt wrote: Hi Marcin! On Sa, 25 Mai 2013, Marcin Szamotulski wrote: I have a proposition for a small but nice feature. Whenever I do diff then diffo (or diffo!) some settings are lost, for example the foldmethod is always reset to manual. It

Has anybody done any work on creating a cmake based build system for Vim?

2013-05-26 Fir de Conversatie Marc Weber
The reason is quite simple - there are so many different ways to build - and reconfiguring takes a lot of time. cmake supports out of source builds - thus you can build mutliple configurations at the same time easily. Marc Weber -- -- You received this message from the vim_dev maillist. Do

python 2.x memory error, bad error message

2013-05-26 Fir de Conversatie Marc Weber
for this case: py vim.current.buffer.append([], 1) if the list is non empty no error happens. Passing an empty list should be a nop. py3 does not show this error. No idea how easy it is to fix. Marc Weber -- -- You received this message from the vim_dev maillist. Do not top-post! Type your

Re: Issue 98 in vim: python.vim indent plugin not pep8 compliant

2013-05-26 Fir de Conversatie vim
Comment #1 on issue 98 by ar...@maven.pl: python.vim indent plugin not pep8 compliant http://code.google.com/p/vim/issues/detail?id=98 Hope to see it in vim 7.4 as there are some The plan is to (1) improve Python support news. -- You received this message because this project is

Re: Feature request: vimdiff and foldmethod

2013-05-26 Fir de Conversatie Gary Johnson
On 2013-05-26, Christian Brabandt wrote: Hi Gary! On Sa, 25 Mai 2013, Gary Johnson wrote: On 2013-05-25, Marcin Szamotulski wrote: Dear Vim Dev, I have a proposition for a small but nice feature. Whenever I do diff then diffo (or diffo!) some settings are lost, for example

Patch 7.3.1029

2013-05-26 Fir de Conversatie Bram Moolenaar
Patch 7.3.1029 Problem:New regexp performance: Unused position state being copied. Solution: Keep track of which positions are actually valid. Files: src/regexp_nfa.c *** ../vim-7.3.1028/src/regexp_nfa.c2013-05-26 21:47:22.0 +0200 --- src/regexp_nfa.c2013-05-26

Patch 7.3.1030

2013-05-26 Fir de Conversatie Bram Moolenaar
Patch 7.3.1030 (after 7.3.1028) Problem:Can't build for debugging. Solution: Fix struct member names. Files: src/regexp_nfa.c *** ../vim-7.3.1029/src/regexp_nfa.c2013-05-26 22:56:16.0 +0200 --- src/regexp_nfa.c2013-05-26 23:12:12.0 +0200 *** ***

Re: Has anybody done any work on creating a cmake based build system for Vim?

2013-05-26 Fir de Conversatie Zulox4
Vim-Cocoa, for OSX system, https://code.google.com/p/vim-cocoa/ has a cmake file. But I think,it is not a good idea to change to. The old make or nmake doesn't change frequently, and it's working well. Also, you can take a look of scons, this page with a list of projects using this building

Re: Has anybody done any work on creating a cmake based build system for Vim?

2013-05-26 Fir de Conversatie Marc Weber
Excerpts from Zulox4's message of Sun May 26 23:54:33 +0200 2013: The old make or nmake doesn't change frequently, and it's working well. 1) configure is slow, if you need to run it once only it doesn't matter. If you have to test multiple cases: If you want to test 4 cases: - no python -

Re: Windows doesn't support -f option

2013-05-26 Fir de Conversatie skeept
On Friday, May 24, 2013 2:15:36 PM UTC-5, skeept wrote: On Friday, May 24, 2013 10:07:09 AM UTC-5, Andrei Olsen wrote: kl. 04:17:42 UTC+2 mandag 20. mai 2013 skrev Ben Fritz følgende: I'm using the Vim without Cream build, version 7.3.822, on Windows 7 64-bit.

Re: Has anybody done any work on creating a cmake based build system for Vim?

2013-05-26 Fir de Conversatie James McCoy
On Mon, May 27, 2013 at 12:38:09AM +0200, Marc Weber wrote: Excerpts from Zulox4's message of Sun May 26 23:54:33 +0200 2013: The old make or nmake doesn't change frequently, and it's working well. 1) configure is slow, if you need to run it once only it doesn't matter. If you have to test

Re: Problems of new regexp engine (that we know)

2013-05-26 Fir de Conversatie mattn
On Friday, May 24, 2013 2:47:18 AM UTC+9, Bram Moolenaar wrote: Please also write tests. Lots of bugs slipped through because we don't have sufficient testing. https://gist.github.com/mattn/5626661 I added test96 -- -- You received this message from the vim_dev maillist. Do not top-post!

Re: Patch 7.3.1021

2013-05-26 Fir de Conversatie John Little
This patch gives me a compile warning from gcc -Wshadow: regexp_nfa.c: In function ‘nfa_regmatch’: regexp_nfa.c:3363:11: warning: declaration of ‘j’ shadows a previous local [-Wshadow] regexp_nfa.c:3002:10: warning: shadowed declaration is here [-Wshadow] int j is used in other cases of the

Re: Has anybody done any work on creating a cmake based build system for Vim?

2013-05-26 Fir de Conversatie Zulox4
On Monday, May 27, 2013 12:38:09 AM UTC+2, MarcWeber wrote: Excerpts from Zulox4's message of Sun May 26 23:54:33 +0200 2013: The old make or nmake doesn't change frequently, and it's working well. 1) configure is slow, if you need to run it once only it doesn't matter. If you have