Re: Proposed patch: NL in sub-replace-expression('\=')

2011-03-24 Fir de Conversatie Motoya Kurotsu
Hi, I attach a patch (against svn revision 1889) and four test files. Some test patterns are recognized as binary by diff. So I attached them separately. The following is a brief explanation of this patch. == Since the patch 7.2.407 was applied, the backslash in the expression is not consumed

Re: Proposed patch: NL in sub-replace-expression('\=')

2011-03-19 Fir de Conversatie Motoya Kurotsu
Hi, Thank for the patch. How about adding a test? I would post the patch as soon as possible. Motoya Kurotsu 2011/3/18 Bram Moolenaar b...@moolenaar.net: Motoya Kurotsu wrote: The patch is attached. Please check and correct. I deleted the following expression in

Re: Proposed patch: NL in sub-replace-expression('\=')

2011-03-18 Fir de Conversatie Bram Moolenaar
Motoya Kurotsu wrote: The patch is attached. Please check and correct. I deleted the following expression in *sub-replace-expression* because the function has been lost since Patch 7.2.437. Prepend a backslash to get a real NL character (which will be a NUL in the file). I added

Re: Proposed patch: NL in sub-replace-expression('\=')

2011-03-16 Fir de Conversatie motz
Hi, The patch is attached. Please check and correct. I deleted the following expression in *sub-replace-expression* because the function has been lost since Patch 7.2.437. Prepend a backslash to get a real NL character (which will be a NUL in the file). I added the description that \=

Re: Proposed patch: NL in sub-replace-expression('\=')

2011-03-16 Fir de Conversatie motz
Hi, The patch is attached. Please check and correct. I deleted the following expression in *sub-replace-expression* because the function has been lost since Patch 7.2.437. Prepend a backslash to get a real NL character (which will be a NUL in the file). I added the description that \=

Re: Proposed patch: NL in sub-replace-expression('\=')

2011-03-16 Fir de Conversatie motz
Please ignore this reply, sorry. On 3月14日, 午前8:42, motz motoya.kuro...@gmail.com wrote: Hi, I would. But English is not my native language. So I hope that someone would check and correct. In Japan or at Tokyo, the periodical blackout is planned from the effect of the earthquake. So it may

Re: Proposed patch: NL in sub-replace-expression('\=')

2011-03-16 Fir de Conversatie Motoya Kurotsu
Hi, The patch is attached. Please check and correct. I deleted the following expression in *sub-replace-expression* because the function has been lost since Patch 7.2.437. Prepend a backslash to get a real NL character (which will be a NUL in the file). I added the description that \=

Re: Proposed patch: NL in sub-replace-expression('\=')

2011-03-13 Fir de Conversatie motz
Hi, I would. But English is not my native language. So I hope that someone would check and correct. In Japan or at Tokyo, the periodical blackout is planned from the effect of the earthquake. So it may take a while to post the patch. But I would do so as possible as I can. Regards, Motoya

Re: Proposed patch: NL in sub-replace-expression('\=')

2011-03-11 Fir de Conversatie Bram Moolenaar
Ben Schmidt wrote: I would like to propose a patch which suppresses the conversion of NL toCR when evaluaation ('\=') is used inside substitute() function. Simply this patch is what makes x == y true in the following code. let x = substitute('a', '.', \n, ) let y =

Re: Proposed patch: NL in sub-replace-expression('\=')

2011-03-09 Fir de Conversatie Ben Schmidt
I would like to propose a patch which suppresses the conversion of NL toCR when evaluaation ('\=') is used inside substitute() function. Simply this patch is what makes x == y true in the following code. let x = substitute('a', '.', \n, ) let y = substitute('a', '.', '\=\n', ) Regardless