On Fri, Aug 31, 2007 at 04:18:57PM +0200, Richard Hartmann wrote:
>
> On 31/08/2007, Tony Mechelynck <antoine.mechelynck@...> wrote:
[...]
> (not as painful as using the GMail interface without fixed width fonts or
> tw=72 which means I need to guess the formatting..) when replying to more
> complex mails.

The MozEx Firefox extension lets you edit textareas with an external editor
(e.g. vim). I'm a fan.

> Richard
--Greg


--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

Milan Vancura wrote:
>> The MozEx Firefox extension lets you edit textareas with an external editor
>> (e.g. vim). I'm a fan.
>
> I've found another FF extension called It's All Text - it allows the same and
> it handles character encoding and cooperates nicely with FF. And it works with
> FF 1.5 and 2.0
>
> Great for editing wiki pages (for example). I can't imagine my work in wiki
> without that extension.
>
> Milan Vancura

Yeah, I use it too. It adds a small blue button labeled "edit" at the bottom
right of textareas when the mouse is over them. Click that button: plop! gvim
opens with any text already typed in. Enter :x to close gvim, plop! the edited
text appears in the browser. Works like magic (as Arthur Weasley would have
said ;-) ).


Best regards,
Tony.
--
The average woman would rather have beauty than brains, because the
average man can see better than he can think.

--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

>
> Indeed. Just out of curiousity, how often do they change between two versions
of
> Vim ?

You can have a look for yourself:

Before I switched to a "Mode":

http://www.vim.org/scripts/script.php?script_id=1353
http://www.vim.org/scripts/script.php?script_id=1548
http://www.vim.org/scripts/script.php?script_id=1547
http://www.vim.org/scripts/script.php?script_id=1554
http://www.vim.org/scripts/script.php?script_id=1565

And later the same 5 plugins as one package:

http://www.vim.org/scripts/script.php?script_id=1609

Especialy when 7.0 with all the new features came out I did 1 to 3
releases per month. Now it is quiter again - but I did a bugfix since
7.1.

Martin


--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

This seems like a quite obvious idea, but why isn't cintent able to
properly indent continuation lines that are outside of a block, even
when the relevant cintent option is set?

Take for instance this perl code:

$foo = 1;
$bar =
     2;

sub alpha {
     return 1;
}

In the above example, cindent would indent:
$bar =
     2;

as:
$bar =
2;

If properly indenting a continuation line outside of a block would be
supported, using cindent to properly indent perl code would be
effective (this is probably useful for other languages as well).


--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

Hi

This thread was in vim_use but I reckon it is more
appropriate to vim_dev now.

First A. Politz  noticed that with the cursor somewhere in
`text' in the example below,

    void fun( /*)*/text())

and with an appropriate function Skip() to reject the `)' in
the comment,

    :call searchpair('(','',')','bcW','Skip()')

would not move the cursor. This is caused by the `c' in the
flags. After finding the `)' in the comment the skip
function rejects it and it matches again. The cursor doesn't
move and do_searchpair() bails out.

And then today,

On 04/09/07, A.Politz <politza@...> wrote:

  > There is another `problem`. The 'r' flag in searchpos()
  > searches for the outermost match, but this searches
  > forever if 'wrapscan' is set and the 'W' flag is not
  > given. Since this is pretty much useless in any case, 'r'
  > therefore should imply 'W'.
  >
  > This is the behaviour at least in this case :
  >
  > :set wrapscan
  >
  > f(a,b,c,d)
  >
  > Cursor on a comma and
  >
  > call searchpair('(',',',')','r')

Or even

    (,)

and use the same call to searchpair('(', ',', ')', 'r')
with the cursor on the `('.

Might the following patch help at all? (I've also included
it as an attachment because I expect gmail to insert
linebreaks, sorry.) I'm not sure about forcing 'W' if 'r' is
in the flags; having 'rw' in flags but getting 'W' behaviour
might cause hard-to-find bugs in somebody's script.
Disallowing the combination might be better. --Antony

*** eval.c.orig Tue Sep 04 22:22:42 2007
--- eval.c Tue Sep 04 22:42:20 2007
***************
*** 14015,14020 ****
--- 14015,14023 ----
  	 options |= SEARCH_START;
       if (flags & SP_END)
  	 options |= SEARCH_END;
+     /* Force 'W' if 'r' is in flags. */
+     if (flags & SP_REPEAT)
+  p_ws = FALSE;

       /* Optional extra argument: line number to stop searching. */
       if (argvars[1].v_type != VAR_UNKNOWN
***************
*** 14275,14280 ****
--- 14278,14287 ----
       {
  	 n = searchit(curwin, curbuf, &pos, dir, pat, 1L,
  					        options, RE_SEARCH, lnum_stop);
+  /* Clear the flag for a match at the starting position: it only makes
+ 	 * sense for the very first search. Otherwise, in conjunction with
+ 	 * the skip function, it can cause an early termination of the loop. */
+  options &= !SEARCH_START;
  	 if (n == FAIL || (firstpos.lnum != 0 && equalpos(pos, firstpos)))
  	     /* didn't find it or found the first match again: FAIL */
  	     break;

--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

Patch for if_perl.xs attached below.

--Suresh


   *** vim7\src\if_perl.xs Tue Sep 04 17:12:39 2007
   --- patched\if_perl.xs Tue Sep 04 16:28:12 2007
   ***************
   *** 40,45 ****
   --- 40,67 ----
     #    define PERL_SUBVERSION SUBVERSION
     #endif

   + /*
   +  * Quoting Jan Dubois of Active State:
   +  *
   +  *    ActivePerl build 822 still identifies itself as 5.8.8 but already
   +  *    contains many of the changes from the upcoming Perl 5.8.9 release.
   +  *
   +  * The changes include addition of two symbols (Perl_sv_2iv_flags,
   +  * Perl_newXS_flags) introduced in not present in earlier releases.
   +  *
   +  * Jan Dubois suggested the following guarding scheme:
   +  *
   +  */
   + #if (ACTIVEPERL_VERSION >= 822)
   + #  define PERL589_OR_LATER
   + #endif
   + #if (PERL_REVISION == 5) && (PERL_VERSION == 8) && (PERL_SUBVERSION >= 9)
   + #  define PERL589_OR_LATER
   + #endif
   + #if (PERL_REVISION == 5) && (PERL_VERSION >= 10)
   + #  define PERL589_OR_LATER
   + #endif
   +
     #ifndef pTHX
     #    define pTHX void
     #    define pTHX_
   ***************
   *** 109,114 ****
   --- 131,140 ----
     # else
     #  define Perl_sv_catpvn dll_Perl_sv_catpvn
     # endif
   + #ifdef PERL589_OR_LATER
   + #  define Perl_sv_2iv_flags    dll_Perl_sv_2iv_flags
   + #  define Perl_newXS_flags     dll_Perl_newXS_flags
   + #endif
     # define Perl_sv_free dll_Perl_sv_free
     # define Perl_sv_isa dll_Perl_sv_isa
     # define Perl_sv_magic dll_Perl_sv_magic
   ***************
   *** 192,197 ****
   --- 218,227 ----
     #else
     static void (*Perl_sv_catpvn)(pTHX_ SV*, const char*, STRLEN);
     #endif
   + #ifdef PERL589_OR_LATER
   + static IV (*Perl_sv_2iv_flags)(pTHX_ SV* sv, I32 flags);
   + static CV * (*Perl_newXS_flags)(pTHX_ const char *name, XSUBADDR_t subaddr,
const char *const filename, const char *const proto,
U32 flags);
   + #endif
     static void (*Perl_sv_free)(pTHX_ SV*);
     static int (*Perl_sv_isa)(pTHX_ SV*, const char*);
     static void (*Perl_sv_magic)(pTHX_ SV*, SV*, int, const char*, I32);
   ***************
   *** 266,271 ****
   --- 296,305 ----
         {"Perl_sv_2pv_nolen", (PERL_PROC*)&Perl_sv_2pv_nolen},
     #else
         {"Perl_sv_2pv", (PERL_PROC*)&Perl_sv_2pv},
   + #endif
   + #ifdef PERL589_OR_LATER
   +     {"Perl_sv_2iv_flags", (PERL_PROC*)&Perl_sv_2iv_flags},
   +     {"Perl_newXS_flags", (PERL_PROC*)&Perl_newXS_flags},
     #endif
         {"Perl_sv_bless", (PERL_PROC*)&Perl_sv_bless},
     #if (PERL_REVISION == 5) && (PERL_VERSION >= 8)



--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

Suresh Govindachar wrote:
>
>
> Patch for if_perl.xs attached below.
>
> --Suresh
>
>
>   *** vim7\src\if_perl.xs Tue Sep 04 17:12:39 2007
>   --- patched\if_perl.xs Tue Sep 04 16:28:12 2007
>   ***************
>   *** 40,45 ****
>   --- 40,67 ----
>     #    define PERL_SUBVERSION SUBVERSION
>     #endif
>
>   + /*
>   +  * Quoting Jan Dubois of Active State:
>   +  *
>   +  *    ActivePerl build 822 still identifies itself as 5.8.8 but already
>   +  *    contains many of the changes from the upcoming Perl 5.8.9 release.
>   +  *
>   +  * The changes include addition of two symbols (Perl_sv_2iv_flags,
>   +  * Perl_newXS_flags) introduced in not present in earlier releases.
>   +  *
>   +  * Jan Dubois suggested the following guarding scheme:
>   +  *
>   +  */
>   + #if (ACTIVEPERL_VERSION >= 822)
>   + #  define PERL589_OR_LATER
>   + #endif
>   + #if (PERL_REVISION == 5) && (PERL_VERSION == 8) && (PERL_SUBVERSION >= 9)
>   + #  define PERL589_OR_LATER
>   + #endif
>   + #if (PERL_REVISION == 5) && (PERL_VERSION >= 10)

---------------------------------------------------^
shouldn't this be (PERL_VERSION >= 9)? Otherwise Perl 5.9.x won't be regarded
as "5.8.9 or later"

>   + #  define PERL589_OR_LATER
>   + #endif
>   +
[...]


Best regards,
Tony.
--
Death is God's way of telling you not to be such a wise guy.

--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

Tony asked:
   > Suresh Govindachar wrote:
   >>
   >> + #if (ACTIVEPERL_VERSION >= 822)
   >> + #  define PERL589_OR_LATER
   >> + #endif
   >> + #if (PERL_REVISION == 5) && (PERL_VERSION == 8) && (PERL_SUBVERSION >= 9)
   >> + #  define PERL589_OR_LATER
   >> + #endif
   >> + #if (PERL_REVISION == 5) && (PERL_VERSION >= 10)
   >
   > -------------------------------------------------^
   > shouldn't this be (PERL_VERSION >= 9)?
   > Otherwise Perl 5.9.x won't be regarded as "5.8.9 or later"

   Good catch but I think there is something about perl's
   version's middle number being even always.  Or it is the
   case that there will not be any 5.9 at all.

   (I heard that both 5.8.9 and 5.10 are expected in a few months
    http://www.slideshare.net/acme/whats-new-in-perl-510 .)

   --Suresh



--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

Suresh Govindachar wrote:
>   Tony asked:
>   > Suresh Govindachar wrote:
>   >>
>   >> + #if (ACTIVEPERL_VERSION >= 822)
>   >> + #  define PERL589_OR_LATER
>   >> + #endif
>   >> + #if (PERL_REVISION == 5) && (PERL_VERSION == 8) && (PERL_SUBVERSION >=
9)
>   >> + #  define PERL589_OR_LATER
>   >> + #endif
>   >> + #if (PERL_REVISION == 5) && (PERL_VERSION >= 10)
>   >
>   > -------------------------------------------------^
>   > shouldn't this be (PERL_VERSION >= 9)?
>   > Otherwise Perl 5.9.x won't be regarded as "5.8.9 or later"
>
>   Good catch but I think there is something about perl's
>   version's middle number being even always.  Or it is the
>   case that there will not be any 5.9 at all.
>
>   (I heard that both 5.8.9 and 5.10 are expected in a few months
>    http://www.slideshare.net/acme/whats-new-in-perl-510 .)
>
>   --Suresh
>
>
>

well, (PERL_VERSION > 8) then? Some versioning schemes use even numbers for
release versions and odd numbers for beta or preliminary versions.

or maybe even

#ifndef PERL_SUBVERSION
# define PERL_SUBVERSION 0
#endif
#if ((PERL_REVISION * 10000) + (PERL_VERSION * 100) + PERL_SUBVERSION >=
50809) || (ACTIVEPERL_VERSION >= 822)
# define PERL589_OR_LATER
#endif


Best regards,
Tony.
--
BE ALERT!!!!  (The world needs more lerts ...)

--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

Andy Wokula schrieb:
> Edward L. Fox schrieb:
>> On 9/2/07, krischik <krischik@...> wrote:
>>> On Sep 2, 4:58 pm, "Richard Hartmann" <richih.mailingl...@...>
>>> wrote:
>>>> On 31/08/07, krischik <krisc...@...> wrote:
>>>>
>>>>> On Aug 30, 2:46 pm, hermi...@... wrote:
>>>>>> PS: As "mode" already means something in Vim context, I do not really
>>>>>> like this new use of the term.
>>>>> Got an alternative name?
>>>> Environment? Framework? Package? Helper?
>>> I like "Framework" best.
>
>> Why not use some geeky expression to make the name *unique*?
>>
>> I suggest calling it "superior".
>
> Bundle?

Oops, this was one click too fast, missed the rest of the thread.

--
Andy

--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

Edward L. Fox schrieb:
> On 9/2/07, krischik <krischik@...> wrote:
>> On Sep 2, 4:58 pm, "Richard Hartmann" <richih.mailingl...@...>
>> wrote:
>>> On 31/08/07, krischik <krisc...@...> wrote:
>>>
>>>> On Aug 30, 2:46 pm, hermi...@... wrote:
>>>>> PS: As "mode" already means something in Vim context, I do not really
>>>>> like this new use of the term.
>>>> Got an alternative name?
>>> Environment? Framework? Package? Helper?
>> I like "Framework" best.

> Why not use some geeky expression to make the name *unique*?
>
> I suggest calling it "superior".

Bundle?

--
Andy

--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

set stl=%!1+1
Ok.

setlocal stl=%!1+1
E539: Illegal character <+>: stl=!%1+1

That doesn't look right,
I believe this could be a appropriate fix :

options.c
6351c6351
<  else if (varp != &p_stl || s[0] != '%' || s[1] != '!')
---
>  else if (gvarp != &p_stl || s[0] != '%' || s[1] != '!')


-ap


--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

> +       /* Clear the flag for a match at the starting position: it only makes
> +        * sense for the very first search. Otherwise, in conjunction with
> +        * the skip function, it can cause an early termination of the loop.
*/
> +       options &= !SEARCH_START;
I believe you ment
              options &= ~SEARCH_START;
>         if (n == FAIL || (firstpos.lnum != 0 && equalpos(pos, firstpos)))
>             /* didn't find it or found the first match again: FAIL */
>             break;
>

-ap


--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

Hello!

Included with v112 of netrw is a new command, :NetrwClean[!] .  What it
does is remove all of netrw's parts from your home directory and, with
the "!" variant, your system directories.   This can be useful when:

* you've put a copy of netrw in your personal directory

* you've now updated your version of vim and it has a more up-to-date
version of netrw that you'd like to be using (usually the personal
directory version overrides the system version).

I've sent a copy of v112 on to Bram, so I expect it'll be showing up on
the ftp site and via cvs Real Soon Now if not already, so some of you
may be faced with this issue.  Thanks, by the way!  I really appreciate
all testing done before I get netrw into some new version of vim where,
if there's some bug still in there, I get inundated with reports.

Anyway, those of you with earlier versions of netrw in their personal
directories can't use the new :NetrwClean command (yet).  So, I've
written a stand-alone version of netrwclean; its available at my website:

http://mysite.verizon.net/astronaut/vim/index.html#NETRWCLEAN

Please feel free to download it and use it.  BTW, it doesn't remove any
vimballs, so netrw.vba[.gz] will be left alone, wherever it may be.

Regards,
Chip Campbell


--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

On 05/09/07, ap <politza@...> wrote:

  >
  >
  > > +       /* Clear the flag for a match at the starting position:
it only makes
  > > +        * sense for the very first search. Otherwise, in conjunction with
  > > +        * the skip function, it can cause an early termination
of the loop. */
  > > +       options &= !SEARCH_START;
  > I believe you ment
  >              options &= ~SEARCH_START;

Definitely! Thanks! --Antony

--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

Patch 7.1.095
Problem:    The FocusLost and FocusGained autocommands are triggered
	     asynchronously in the GUI.  This may cause arbitrary problems.
Solution:   Put the focus event in the input buffer and handle it when ready
	     for it.
Files:     src/eval.c, src/getchar.c, src/gui.c, src/gui_gtk_x11.c,
	     src/keymap.h


*** ../vim-7.1.094/src/eval.c Thu Aug 30 11:10:38 2007
--- src/eval.c Mon Sep  3 22:48:09 2007
***************
*** 9912,9929 ****

       ++no_mapping;
       ++allow_keys;
!     if (argvars[0].v_type == VAR_UNKNOWN)
!  /* getchar(): blocking wait. */
!  n = safe_vgetc();
!     else if (get_tv_number_chk(&argvars[0], &error) == 1)
!  /* getchar(1): only check if char avail */
!  n = vpeekc();
!     else if (error || vpeekc() == NUL)
!  /* illegal argument or getchar(0) and no char avail: return zero */
!  n = 0;
!     else
!  /* getchar(0) and char avail: return char */
!  n = safe_vgetc();
       --no_mapping;
       --allow_keys;

--- 9912,9935 ----

       ++no_mapping;
       ++allow_keys;
!     for (;;)
!     {
!  if (argvars[0].v_type == VAR_UNKNOWN)
! 	    /* getchar(): blocking wait. */
! 	    n = safe_vgetc();
!  else if (get_tv_number_chk(&argvars[0], &error) == 1)
! 	    /* getchar(1): only check if char avail */
! 	    n = vpeekc();
!  else if (error || vpeekc() == NUL)
! 	    /* illegal argument or getchar(0) and no char avail: return zero */
! 	    n = 0;
!  else
! 	    /* getchar(0) and char avail: return char */
! 	    n = safe_vgetc();
!  if (n == K_IGNORE)
! 	    continue;
!  break;
!     }
       --no_mapping;
       --allow_keys;

*** ../vim-7.1.094/src/getchar.c Thu May 10 18:43:02 2007
--- src/getchar.c Wed Aug 29 22:38:49 2007
***************
*** 1596,1603 ****
  		 continue;
  	     }
   #endif
-
   #ifdef FEAT_GUI
  	     /* Translate K_CSI to CSI.  The special key is only used to avoid
  	      * it being recognized as the start of a special key. */
  	     if (c == K_CSI)
--- 1596,1610 ----
  		 continue;
  	     }
   #endif
   #ifdef FEAT_GUI
+ 	    /* The caller doesn't need to know that the focus event is delayed
+ 	     * until getting a character. */
+ 	    if (c == K_FOCUSGAINED || c == K_FOCUSLOST)
+ 	    {
+ 	 ui_focus_change(c == K_FOCUSGAINED);
+ 	 continue;
+ 	    }
+
  	     /* Translate K_CSI to CSI.  The special key is only used to avoid
  	      * it being recognized as the start of a special key. */
  	     if (c == K_CSI)
*** ../vim-7.1.094/src/gui.c Thu Aug 30 13:51:52 2007
--- src/gui.c Thu Aug 30 14:10:48 2007
***************
*** 4519,4525 ****
       xim_set_focus(in_focus);
   # endif

!     ui_focus_change(in_focus);
   #endif
   }

--- 4519,4536 ----
       xim_set_focus(in_focus);
   # endif

!     /* Put events in the input queue only when allowed.
!      * ui_focus_change() isn't called directly, because it invokes
!      * autocommands and that must not happen asynchronously. */
!     if (!hold_gui_events)
!     {
!  char_u  bytes[3];
!
!  bytes[0] = CSI;
!  bytes[1] = KS_EXTRA;
!  bytes[2] = in_focus ? (int)KE_FOCUSGAINED : (int)KE_FOCUSLOST;
!  add_to_input_buf(bytes, 3);
!     }
   #endif
   }

*** ../vim-7.1.094/src/gui_gtk_x11.c Tue Jun 19 18:07:52 2007
--- src/gui_gtk_x11.c Wed Aug 29 22:43:34 2007
***************
*** 813,822 ****
       if (blink_state == BLINK_NONE)
  	 gui_mch_start_blink();

!     /* make sure keyboard input goes to the draw area (if this is focus for a
window) */
       if (widget != gui.drawarea)
  	 gtk_widget_grab_focus(gui.drawarea);

       return TRUE;
   }

--- 813,827 ----
       if (blink_state == BLINK_NONE)
  	 gui_mch_start_blink();

!     /* make sure keyboard input goes to the draw area (if this is focus for a
!      * window) */
       if (widget != gui.drawarea)
  	 gtk_widget_grab_focus(gui.drawarea);

+     /* make sure the input buffer is read */
+     if (gtk_main_level() > 0)
+  gtk_main_quit();
+
       return TRUE;
   }

***************
*** 828,833 ****
--- 833,842 ----

       if (blink_state != BLINK_NONE)
  	 gui_mch_stop_blink();
+
+     /* make sure the input buffer is read */
+     if (gtk_main_level() > 0)
+  gtk_main_quit();

       return TRUE;
   }
*** ../vim-7.1.094/src/keymap.h Sat May  5 19:34:22 2007
--- src/keymap.h Wed Aug 29 22:17:51 2007
***************
*** 254,259 ****
--- 254,261 ----
       , KE_DROP  /* DnD data is available */
       , KE_CURSORHOLD /* CursorHold event */
       , KE_NOP  /* doesn't do something */
+     , KE_FOCUSGAINED /* focus gained */
+     , KE_FOCUSLOST /* focus lost */
   };

   /*
***************
*** 445,450 ****
--- 447,454 ----
   #define K_CMDWIN TERMCAP2KEY(KS_EXTRA, KE_CMDWIN)

   #define K_DROP  TERMCAP2KEY(KS_EXTRA, KE_DROP)
+ #define K_FOCUSGAINED TERMCAP2KEY(KS_EXTRA, KE_FOCUSGAINED)
+ #define K_FOCUSLOST TERMCAP2KEY(KS_EXTRA, KE_FOCUSLOST)

   #define K_CURSORHOLD TERMCAP2KEY(KS_EXTRA, KE_CURSORHOLD)

*** ../vim-7.1.094/src/version.c Thu Aug 30 19:36:52 2007
--- src/version.c Wed Sep  5 21:42:41 2007
***************
*** 668,669 ****
--- 668,671 ----
   {   /* Add new patch number below this line */
+ /**/
+     95,
   /**/

--
ARTHUR:      Who are you?
TALL KNIGHT: We are the Knights Who Say "Ni"!
BEDEVERE:    No!  Not the Knights Who Say "Ni"!
                  "Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD

  /// Bram Moolenaar -- Bram@... -- http://www.Moolenaar.net   \\\
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\        download, build and distribute -- http://www.A-A-P.org        ///
  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

Politza wrote:

> set stl=%!1+1
> Ok.
>
> setlocal stl=%!1+1
> E539: Illegal character <+>: stl=!%1+1
>
> That doesn't look right,
> I believe this could be a appropriate fix :
>
> options.c
> 6351c6351
> <  else if (varp != &p_stl || s[0] != '%' || s[1] != '!')
> ---
> >  else if (gvarp != &p_stl || s[0] != '%' || s[1] != '!')

Thanks!

--
FATHER:       Make sure the Prince doesn't leave this room until I come and
               get him.
FIRST GUARD:  Not ... to leave the room ... even if you come and get him.
FATHER:       No.  Until I come and get him.
SECOND GUARD: Hic.
                  "Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD

  /// Bram Moolenaar -- Bram@... -- http://www.Moolenaar.net   \\\
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\        download, build and distribute -- http://www.A-A-P.org        ///
  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

Suresh Govindachar wrote:

> Patch for if_perl.xs attached below.

Thanks!

Let me know if someone experiences a problem with this.

--
"Never be afraid to tell the world who you are."
                                         -- Anonymous

  /// Bram Moolenaar -- Bram@... -- http://www.Moolenaar.net   \\\
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\        download, build and distribute -- http://www.A-A-P.org        ///
  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

Charles E Campbell Jr wrote:
> Hello!
>
> Included with v112 of netrw is a new command, :NetrwClean[!] .  What it
> does is remove all of netrw's parts from your home directory and, with
> the "!" variant, your system directories.   This can be useful when:
>
> * you've put a copy of netrw in your personal directory
>
> * you've now updated your version of vim and it has a more up-to-date
> version of netrw that you'd like to be using (usually the personal
> directory version overrides the system version).
>
> I've sent a copy of v112 on to Bram, so I expect it'll be showing up on
> the ftp site and via cvs Real Soon Now if not already, so some of you
> may be faced with this issue.  Thanks, by the way!  I really appreciate
> all testing done before I get netrw into some new version of vim where,
> if there's some bug still in there, I get inundated with reports.
>
> Anyway, those of you with earlier versions of netrw in their personal
> directories can't use the new :NetrwClean command (yet).  So, I've
> written a stand-alone version of netrwclean; its available at my website:
>
> http://mysite.verizon.net/astronaut/vim/index.html#NETRWCLEAN
>
> Please feel free to download it and use it.  BTW, it doesn't remove any
> vimballs, so netrw.vba[.gz] will be left alone, wherever it may be.
>
> Regards,
> Chip Campbell

At the moment there is still a spurious runtime/plugin/netrw.vim on the rsync
server, which is (IIUC) an obsolete version of autoload/netrw.vim. Let's hope
it will disappear when the above will appear (presumably autoload/netrw.vim
and/or plugin/netrwPlugin.vim etc.).


Best regards,
Tony.
--
No matter how subtle the wizard, a knife in the shoulder blades will
seriously cramp his style.

--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

Patch 7.1.096
Problem:    Reading past end of a string when resizing Vim. (Dominique Pelle)
Solution:   Check the string pointer before getting the char it points to.
Files:     src/message.c


*** ../vim-7.1.095/src/message.c Tue Aug  7 21:59:26 2007
--- src/message.c Thu Aug 30 22:53:03 2007
***************
*** 944,949 ****
--- 944,950 ----
  		 c = K_IGNORE;
  	     }
   #endif
+
  	     /*
  	      * Allow scrolling back in the messages.
  	      * Also accept scroll-down commands when messages fill the screen,
***************
*** 1840,1845 ****
--- 1841,1847 ----
       char_u *sb_str = str;
       int  sb_col = msg_col;
       int  wrap;
+     int  did_last_char;

       did_wait_return = FALSE;
       while ((maxlen < 0 || (int)(s - str) < maxlen) && *s != NUL)
***************
*** 1909,1915 ****
--- 1911,1920 ----
  		 else
   #endif
  		     msg_screen_putchar(*s++, attr);
+ 	 did_last_char = TRUE;
  	     }
+ 	    else
+ 	 did_last_char = FALSE;

  	     if (p_more)
  		 /* store text for scrolling back */
***************
*** 1944,1954 ****

  	     /* When we displayed a char in last column need to check if there
  	      * is still more. */
! 	    if (*s >= ' '
! #ifdef FEAT_RIGHTLEFT
! 		    && !cmdmsg_rl
! #endif
! 	       )
  		 continue;
  	 }

--- 1949,1955 ----

  	     /* When we displayed a char in last column need to check if there
  	      * is still more. */
! 	    if (did_last_char)
  		 continue;
  	 }

*** ../vim-7.1.095/src/version.c Wed Sep  5 21:45:54 2007
--- src/version.c Thu Sep  6 12:31:28 2007
***************
*** 668,669 ****
--- 668,671 ----
   {   /* Add new patch number below this line */
+ /**/
+     96,
   /**/

--
Yah, well, we had to carve our electrons out of driftwood we'd
find.  In the winter.  Uphill.  Both ways.

  /// Bram Moolenaar -- Bram@... -- http://www.Moolenaar.net   \\\
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\        download, build and distribute -- http://www.A-A-P.org        ///
  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

Tony Mechelynck wrote:

> At the moment there is still a spurious runtime/plugin/netrw.vim on
> the rsync server, which is (IIUC) an obsolete version of
> autoload/netrw.vim. Let's hope it will disappear when the above will
> appear (presumably autoload/netrw.vim and/or plugin/netrwPlugin.vim
> etc.).

Oh, that netrw.vim should be in the autoload directory instead of in
the plugin directory.  Must be a mistake I made when copying files of a
new version.  I'll correct it now.

--
Luxury. We used to have to get out of the lake at three o'clock in the
morning, clean the lake, eat a handful of hot gravel, go to work at the
mill every day for tuppence a month, come home, and Dad would beat us
around the head and neck with a broken bottle, if we were LUCKY!

  /// Bram Moolenaar -- Bram@... -- http://www.Moolenaar.net   \\\
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\        download, build and distribute -- http://www.A-A-P.org        ///
  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

Patch 7.1.097
Problem:    ":setlocal stl=%!1+1" does not work.
Solution:   Adjust check for pointer. (Politz)
Files:     src/option.c


*** ../vim-7.1.096/src/option.c Sun Aug 12 15:50:26 2007
--- src/option.c Wed Sep  5 22:34:27 2007
***************
*** 6348,6354 ****
  		 errmsg = check_stl_option(p_ruf);
  	 }
  	 /* check 'statusline' only if it doesn't start with "%!" */
!  else if (varp != &p_stl || s[0] != '%' || s[1] != '!')
  	     errmsg = check_stl_option(s);
  	 if (varp == &p_ruf && errmsg == NULL)
  	     comp_col();
--- 6352,6358 ----
  		 errmsg = check_stl_option(p_ruf);
  	 }
  	 /* check 'statusline' only if it doesn't start with "%!" */
!  else if (varp == &p_ruf || s[0] != '%' || s[1] != '!')
  	     errmsg = check_stl_option(s);
  	 if (varp == &p_ruf && errmsg == NULL)
  	     comp_col();
*** ../vim-7.1.096/src/version.c Thu Sep  6 12:53:59 2007
--- src/version.c Thu Sep  6 13:31:37 2007
***************
*** 668,669 ****
--- 668,671 ----
   {   /* Add new patch number below this line */
+ /**/
+     97,
   /**/

--
Not too long ago, cut and paste was done with scissors and glue...

  /// Bram Moolenaar -- Bram@... -- http://www.Moolenaar.net   \\\
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\        download, build and distribute -- http://www.A-A-P.org        ///
  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

On 9/4/07, Bram Moolenaar <Bram@...> wrote:
>
>
> Hello Vim users,
>
> So far we only had the possibility to search for scripts on the Vim
> website.  Of course you could use your favorite search engine, but then
> you would find any type of file, not just Vim scripts.
>
> I have now added a search box to the search page on www.vim.org where
> you can specifically search for Vim scripts, anywhere on the internet.
> This uses Google Code Search.  It not only finds individual "*.vim"
> files but also scripts inside archives.  E.g., in the Vim distribution.
>
> There is one extra special thing: You search with a regexp pattern.
> Thus what you type in the search box are not keywords, it is different
> from a normal Google search.  For example, you can search for
> "python.*indent" and find matches in "Python indent", "GetPythonIndent"
> and "python_highlight_indent_errors".  The regexp synax used is Posix,
> it's different from Vim regexp.
>
> Try it out on the Vim search page (second box from the top):
>        http://www.vim.org/search.php

Wow, regexes in in google codesearch.
Is it what you added ? Do they plan to regex-search anywhere :-) ?

Yakov

--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

Patch 7.1.098
Problem:    ":call s:var()" doesn't work if "s:var" is a Funcref. (Andy Wokula)
Solution:   Before converting "s:" into a script ID, check if it is a Funcref.
Files:     src/eval.c


*** ../vim-7.1.097/src/eval.c Wed Sep  5 21:45:54 2007
--- src/eval.c Thu Sep  6 12:11:19 2007
***************
*** 19367,19372 ****
--- 19367,19394 ----
       if (lv.ll_name == NULL)
       {
  	 /* Error found, but continue after the function name. */
+  *pp = end;
+  goto theend;
+     }
+
+     /* Check if the name is a Funcref.  If so, use the value. */
+     if (lv.ll_exp_name != NULL)
+     {
+  len = (int)STRLEN(lv.ll_exp_name);
+  name = deref_func_name(lv.ll_exp_name, &len);
+  if (name == lv.ll_exp_name)
+ 	    name = NULL;
+     }
+     else
+     {
+  len = (int)(end - *pp);
+  name = deref_func_name(*pp, &len);
+  if (name == *pp)
+ 	    name = NULL;
+     }
+     if (name != NULL)
+     {
+  name = vim_strsave(name);
  	 *pp = end;
  	 goto theend;
       }
*** ../vim-7.1.097/src/version.c Thu Sep  6 13:32:53 2007
--- src/version.c Thu Sep  6 14:24:10 2007
***************
*** 668,669 ****
--- 668,671 ----
   {   /* Add new patch number below this line */
+ /**/
+     98,
   /**/

--
Not too long ago, a program was something you watched on TV...

  /// Bram Moolenaar -- Bram@... -- http://www.Moolenaar.net   \\\
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\        download, build and distribute -- http://www.A-A-P.org        ///
  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

Bram Moolenaar wrote:
> Tony Mechelynck wrote:
>
>> At the moment there is still a spurious runtime/plugin/netrw.vim on
>> the rsync server, which is (IIUC) an obsolete version of
>> autoload/netrw.vim. Let's hope it will disappear when the above will
>> appear (presumably autoload/netrw.vim and/or plugin/netrwPlugin.vim
>> etc.).
>
> Oh, that netrw.vim should be in the autoload directory instead of in
> the plugin directory.  Must be a mistake I made when copying files of a
> new version.  I'll correct it now.
>

rsync has just downloaded (among others) autoload/netrw.vim but it is v110,
not the v112 announced by Dr. Chip at the start of this thread. I hope nothing
went wrong?


Best regards,
Tony.
--
Bees are very busy souls
They have no time for birth controls
And that is why in times like these
There are so many Sons of Bees.

--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

Patch 7.1.099
Problem:    When the 'keymap' and 'paste' options have a non-default value,
	     ":mkexrc" and ":mksession" do not correctly set the options.
Solution:   Set the options with side effects before other options.
Files:     src/option.c


*** ../vim-7.1.098/src/option.c Thu Sep  6 13:32:53 2007
--- src/option.c Wed Sep  5 22:34:27 2007
***************
*** 427,432 ****
--- 427,434 ----
   #define P_NOGLOB       0x100000L/* do not use local value for global vimrc */
   #define P_NFNAME       0x200000L/* only normal file name chars allowed */
   #define P_INSECURE     0x400000L/* option was set from a modeline */
+ #define P_PRI_MKRC     0x800000L/* priority for :mkvimrc (setting option has
+ 				   side effects) */

   #define ISK_LATIN1  (char_u *)"@,48-57,_,192-255"

***************
*** 773,778 ****
--- 775,782 ----
  			     {(char_u *)0L, (char_u *)0L}
   #endif
  			     },
+ 			    /* P_PRI_MKRC isn't needed here, optval_default()
+ 			     * always returns TRUE for 'compatible' */
       {"compatible",  "cp",   P_BOOL|P_RALL,
  			     (char_u *)&p_cp, PV_NONE,
  			     {(char_u *)TRUE, (char_u *)FALSE}},
***************
*** 1515,1521 ****
  			     {(char_u *)0L, (char_u *)0L}
   #endif
  			     },
!     {"keymap",     "kmp", 
P_STRING|P_ALLOCED|P_VI_DEF|P_RBUF|P_RSTAT|P_NFNAME,
   #ifdef FEAT_KEYMAP
  			     (char_u *)&p_keymap, PV_KMAP,
  			     {(char_u *)"", (char_u *)0L}
--- 1519,1525 ----
  			     {(char_u *)0L, (char_u *)0L}
   #endif
  			     },
!     {"keymap",     "kmp", 
P_STRING|P_ALLOCED|P_VI_DEF|P_RBUF|P_RSTAT|P_NFNAME|P_PRI_MKRC,
   #ifdef FEAT_KEYMAP
  			     (char_u *)&p_keymap, PV_KMAP,
  			     {(char_u *)"", (char_u *)0L}
***************
*** 1836,1842 ****
       {"paragraphs",  "para", P_STRING|P_VI_DEF,
  			     (char_u *)&p_para, PV_NONE,
  			     {(char_u *)"IPLPPPQPP LIpplpipbp", (char_u *)0L}},
!     {"paste",     NULL,   P_BOOL|P_VI_DEF,
  			     (char_u *)&p_paste, PV_NONE,
  			     {(char_u *)FALSE, (char_u *)0L}},
       {"pastetoggle", "pt",   P_STRING|P_VI_DEF,
--- 1840,1846 ----
       {"paragraphs",  "para", P_STRING|P_VI_DEF,
  			     (char_u *)&p_para, PV_NONE,
  			     {(char_u *)"IPLPPPQPP LIpplpipbp", (char_u *)0L}},
!     {"paste",     NULL,   P_BOOL|P_VI_DEF|P_PRI_MKRC,
  			     (char_u *)&p_paste, PV_NONE,
  			     {(char_u *)FALSE, (char_u *)0L}},
       {"pastetoggle", "pt",   P_STRING|P_VI_DEF,
***************
*** 8535,8547 ****
       char_u  *varp_local = NULL; /* fresh value */
       char  *cmd;
       int 	 round;

       /*
        * The options that don't have a default (terminal name, columns, lines)
        * are never written.  Terminal options are also not written.
        */
!     for (p = &options[0]; !istermoption(p); p++)
!  if (!(p->flags & P_NO_MKRC) && !istermoption(p))
  	 {
  	     /* skip global option when only doing locals */
  	     if (p->indir == PV_NONE && !(opt_flags & OPT_GLOBAL))
--- 8539,8558 ----
       char_u  *varp_local = NULL; /* fresh value */
       char  *cmd;
       int 	 round;
+     int 	 pri;

       /*
        * The options that don't have a default (terminal name, columns, lines)
        * are never written.  Terminal options are also not written.
+      * Do the loop over "options[]" twice: once for options with the
+      * P_PRI_MKRC flag and once without.
        */
!     for (pri = 1; pri >= 0; --pri)
!     {
!       for (p = &options[0]; !istermoption(p); p++)
!  if (!(p->flags & P_NO_MKRC)
! 	 && !istermoption(p)
! 	 && ((pri == 1) == ((p->flags & P_PRI_MKRC) != 0)))
  	 {
  	     /* skip global option when only doing locals */
  	     if (p->indir == PV_NONE && !(opt_flags & OPT_GLOBAL))
***************
*** 8637,8642 ****
--- 8648,8654 ----
  		 }
  	     }
  	 }
+     }
       return OK;
   }

*** ../vim-7.1.098/src/version.c Thu Sep  6 14:25:50 2007
--- src/version.c Thu Sep  6 16:32:31 2007
***************
*** 668,669 ****
--- 668,671 ----
   {   /* Add new patch number below this line */
+ /**/
+     99,
   /**/

--
Not too long ago, a keyboard was something to make music with...

  /// Bram Moolenaar -- Bram@... -- http://www.Moolenaar.net   \\\
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\        download, build and distribute -- http://www.A-A-P.org        ///
  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

Patch 7.1.100
Problem:    Win32: Executing cscope doesn't always work properly.
Solution:   Use another way to invoke cscope. (Mike Williams)
Files:     src/if_cscope.c, src/if_cscope.h, src/main.c,
	     src/proto/if_cscope.pro


*** ../vim-7.1.099/src/if_cscope.c Tue Aug 21 18:02:58 2007
--- src/if_cscope.c Sun Sep  2 16:50:50 2007
***************
*** 24,34 ****
       /* not UNIX, must be WIN32 */
   # include "vimio.h"
   # include <fcntl.h>
- # include <process.h>
- # define STDIN_FILENO    0
- # define STDOUT_FILENO   1
- # define STDERR_FILENO   2
- # define pipe(fds) _pipe(fds, 256, O_TEXT|O_NOINHERIT)
   #endif
   #include "if_cscope.h"

--- 24,29 ----
***************
*** 65,71 ****
   static char *     cs_parse_results __ARGS((int cnumber, char *buf, int
bufsize, char **context, char **linenumber, char **search));
   static char *     cs_pathcomponents __ARGS((char *path));
   static void     cs_print_tags_priv __ARGS((char **, char **, int));
! static int     cs_read_prompt __ARGS((int ));
   static void     cs_release_csp __ARGS((int, int freefnpp));
   static int     cs_reset __ARGS((exarg_T *eap));
   static char *     cs_resolve_file __ARGS((int, char *));
--- 60,66 ----
   static char *     cs_parse_results __ARGS((int cnumber, char *buf, int
bufsize, char **context, char **linenumber, char **search));
   static char *     cs_pathcomponents __ARGS((char *path));
   static void     cs_print_tags_priv __ARGS((char **, char **, int));
! static int     cs_read_prompt __ARGS((int));
   static void     cs_release_csp __ARGS((int, int freefnpp));
   static int     cs_reset __ARGS((exarg_T *eap));
   static char *     cs_resolve_file __ARGS((int, char *));
***************
*** 504,510 ****
   #if defined(UNIX)
       else if (S_ISREG(statbuf.st_mode) || S_ISLNK(statbuf.st_mode))
   #else
!  /* substitute define S_ISREG from os_unix.h */
       else if (((statbuf.st_mode) & S_IFMT) == S_IFREG)
   #endif
       {
--- 499,505 ----
   #if defined(UNIX)
       else if (S_ISREG(statbuf.st_mode) || S_ISLNK(statbuf.st_mode))
   #else
!  /* WIN32 - substitute define S_ISREG from os_unix.h */
       else if (((statbuf.st_mode) & S_IFMT) == S_IFREG)
   #endif
       {
***************
*** 717,733 ****
   cs_create_connection(i)
       int i;
   {
!     int to_cs[2], from_cs[2], len;
!     char *prog, *cmd, *ppath = NULL;
! #ifndef UNIX
!     int in_save, out_save, err_save;
!     long_i ph;
! # ifdef FEAT_GUI
!     HWND activewnd = NULL;
!     HWND consolewnd = NULL;
! # endif
   #endif

       /*
        * Cscope reads from to_cs[0] and writes to from_cs[1]; vi reads from
        * from_cs[0] and writes to to_cs[1].
--- 712,734 ----
   cs_create_connection(i)
       int i;
   {
! #ifdef UNIX
!     int  to_cs[2], from_cs[2];
! #endif
!     int  len;
!     char *prog, *cmd, *ppath = NULL;
! #ifdef WIN32
!     int  fd;
!     SECURITY_ATTRIBUTES sa;
!     PROCESS_INFORMATION pi;
!     STARTUPINFO si;
!     BOOL pipe_stdin = FALSE, pipe_stdout = FALSE;
!     HANDLE stdin_rd, stdout_rd;
!     HANDLE stdout_wr, stdin_wr;
!     BOOL created;
   #endif

+ #if defined(UNIX)
       /*
        * Cscope reads from to_cs[0] and writes to from_cs[1]; vi reads from
        * from_cs[0] and writes to to_cs[1].
***************
*** 748,765 ****
  	 return CSCOPE_FAILURE;
       }

- #if defined(UNIX)
       switch (csinfo[i].pid = fork())
       {
       case -1:
  	 (void)EMSG(_("E622: Could not fork for cscope"));
  	 goto err_closing;
       case 0: 		 /* child: run cscope. */
- #else
-  in_save = dup(STDIN_FILENO);
-  out_save = dup(STDOUT_FILENO);
-  err_save = dup(STDERR_FILENO);
- #endif
  	 if (dup2(to_cs[0], STDIN_FILENO) == -1)
  	     PERROR("cs_create_connection 1");
  	 if (dup2(from_cs[1], STDOUT_FILENO) == -1)
--- 749,760 ----
***************
*** 768,782 ****
  	     PERROR("cs_create_connection 3");

  	 /* close unused */
- #if defined(UNIX)
  	 (void)close(to_cs[1]);
  	 (void)close(from_cs[0]);
   #else
!  /* On win32 we must close opposite ends because we are the parent */
!  (void)close(to_cs[0]);
!  to_cs[0] = -1;
!  (void)close(from_cs[1]);
!  from_cs[1] = -1;
   #endif
  	 /* expand the cscope exec for env var's */
  	 if ((prog = (char *)alloc(MAXPATHL + 1)) == NULL)
--- 763,794 ----
  	     PERROR("cs_create_connection 3");

  	 /* close unused */
  	 (void)close(to_cs[1]);
  	 (void)close(from_cs[0]);
   #else
!  /* WIN32 */
!  /* Create pipes to communicate with cscope */
!  sa.nLength = sizeof(SECURITY_ATTRIBUTES);
!  sa.bInheritHandle = TRUE;
!  sa.lpSecurityDescriptor = NULL;
!
!  if (!(pipe_stdin = CreatePipe(&stdin_rd, &stdin_wr, &sa, 0))
! 	 || !(pipe_stdout = CreatePipe(&stdout_rd, &stdout_wr, &sa, 0)))
!  {
! 	    (void)EMSG(_("E566: Could not create cscope pipes"));
! err_closing:
! 	    if (pipe_stdin)
! 	    {
! 	 CloseHandle(stdin_rd);
! 	 CloseHandle(stdin_wr);
! 	    }
! 	    if (pipe_stdout)
! 	    {
! 	 CloseHandle(stdout_rd);
! 	 CloseHandle(stdout_wr);
! 	    }
! 	    return CSCOPE_FAILURE;
!  }
   #endif
  	 /* expand the cscope exec for env var's */
  	 if ((prog = (char *)alloc(MAXPATHL + 1)) == NULL)
***************
*** 784,789 ****
--- 796,802 ----
   #ifdef UNIX
  	     return CSCOPE_FAILURE;
   #else
+ 	    /* WIN32 */
  	     goto err_closing;
   #endif
  	 }
***************
*** 800,805 ****
--- 813,819 ----
   #ifdef UNIX
  		 return CSCOPE_FAILURE;
   #else
+ 	 /* WIN32 */
  		 goto err_closing;
   #endif
  	     }
***************
*** 818,823 ****
--- 832,838 ----
   #ifdef UNIX
  	     return CSCOPE_FAILURE;
   #else
+ 	    /* WIN32 */
  	     goto err_closing;
   #endif
  	 }
***************
*** 826,831 ****
--- 841,847 ----
   #if defined(UNIX)
  	 (void)sprintf(cmd, "exec %s -dl -f %s", prog, csinfo[i].fname);
   #else
+  /* WIN32 */
  	 (void)sprintf(cmd, "%s -dl -f %s", prog, csinfo[i].fname);
   #endif
  	 if (csinfo[i].ppath != NULL)
***************
*** 851,910 ****
  	 exit(127);
  	 /* NOTREACHED */
       default: /* parent. */
- #else
- # ifdef FEAT_GUI
-  activewnd = GetForegroundWindow(); /* on win9x cscope steals focus */
-  /* Dirty hack to hide annoying console window */
-  if (AllocConsole())
-  {
- 	    char *title;
- 	    title = (char *)alloc(1024);
- 	    if (title == NULL)
- 	 FreeConsole();
- 	    else
- 	    {
- 	 GetConsoleTitle(title, 1024); /* save for future restore */
- 	 SetConsoleTitle(
- 		    "GVIMCS{5499421B-CBEF-45b0-85EF-38167FDEA5C5}GVIMCS");
- 	 Sleep(40); /* as stated in MS KB we must wait 40 ms */
- 	 consolewnd = FindWindow(NULL,
- 		 "GVIMCS{5499421B-CBEF-45b0-85EF-38167FDEA5C5}GVIMCS");
- 	 if (consolewnd != NULL)
- 		    ShowWindow(consolewnd, SW_HIDE);
- 	 SetConsoleTitle(title);
- 	 vim_free(title);
- 	    }
-  }
- # endif
-  /* May be use &shell, &shellquote etc */
- # ifdef __BORLANDC__
-  /* BCC 5.5 uses a different function name for spawnlp */
-  ph = (long_i)spawnlp(P_NOWAIT, prog, cmd, NULL);
- # else
-  ph = (long_i)_spawnlp(_P_NOWAIT, prog, cmd, NULL);
- # endif
-  vim_free(prog);
-  vim_free(cmd);
- # ifdef FEAT_GUI
-  /* Dirty hack part two */
-  if (activewnd != NULL)
- 	    /* restoring focus */
- 	    SetForegroundWindow(activewnd);
-  if (consolewnd != NULL)
- 	    FreeConsole();
-
- # endif
-  if (ph == -1)
-  {
- 	    PERROR(_("cs_create_connection exec failed"));
- 	    (void)EMSG(_("E623: Could not spawn cscope process"));
- 	    goto err_closing;
-  }
-  /* else */
-  csinfo[i].pid = 0;
-  csinfo[i].hProc = (HANDLE)ph;
-
- #endif /* !UNIX */
  	 /*
  	  * Save the file descriptors for later duplication, and
  	  * reopen as streams.
--- 867,872 ----
***************
*** 914,935 ****
  	 if ((csinfo[i].fr_fp = fdopen(from_cs[0], "r")) == NULL)
  	     PERROR(_("cs_create_connection: fdopen for fr_fp failed"));

- #if defined(UNIX)
  	 /* close unused */
  	 (void)close(to_cs[0]);
  	 (void)close(from_cs[1]);

  	 break;
       }
   #else
!  /* restore stdhandles */
!     dup2(in_save, STDIN_FILENO);
!     dup2(out_save, STDOUT_FILENO);
!     dup2(err_save, STDERR_FILENO);
!     close(in_save);
!     close(out_save);
!     close(err_save);
! #endif
       return CSCOPE_SUCCESS;
   } /* cs_create_connection */

--- 876,927 ----
  	 if ((csinfo[i].fr_fp = fdopen(from_cs[0], "r")) == NULL)
  	     PERROR(_("cs_create_connection: fdopen for fr_fp failed"));

  	 /* close unused */
  	 (void)close(to_cs[0]);
  	 (void)close(from_cs[1]);

  	 break;
       }
+
   #else
!     /* WIN32 */
!     /* Create a new process to run cscope and use pipes to talk with it */
!     GetStartupInfo(&si);
!     si.dwFlags = STARTF_USESTDHANDLES | STARTF_USESHOWWINDOW;
!     si.wShowWindow = SW_HIDE;  /* Hide child application window */
!     si.hStdOutput = stdout_wr;
!     si.hStdError  = stdout_wr;
!     si.hStdInput  = stdin_rd;
!     created = CreateProcess(NULL, cmd, NULL, NULL, TRUE, CREATE_NEW_CONSOLE,
! 						 NULL, NULL, &si, &pi);
!     vim_free(prog);
!     vim_free(cmd);
!
!     if (!created)
!     {
!  PERROR(_("cs_create_connection exec failed"));
!  (void)EMSG(_("E623: Could not spawn cscope process"));
!  goto err_closing;
!     }
!     /* else */
!     csinfo[i].pid = pi.dwProcessId;
!     csinfo[i].hProc = pi.hProcess;
!     CloseHandle(pi.hThread);
!
!     /* TODO - tidy up after failure to create files on pipe handles. */
!     if (((fd = _open_osfhandle((intptr_t)stdin_wr, _O_TEXT|_O_APPEND)) < 0)
! 	    || ((csinfo[i].to_fp = _fdopen(fd, "w")) == NULL))
!  PERROR(_("cs_create_connection: fdopen for to_fp failed"));
!     if (((fd = _open_osfhandle((intptr_t)stdout_rd, _O_TEXT|_O_RDONLY)) < 0)
! 	    || ((csinfo[i].fr_fp = _fdopen(fd, "r")) == NULL))
!  PERROR(_("cs_create_connection: fdopen for fr_fp failed"));
!
!     /* Close handles for file descriptors inherited by the cscope process */
!     CloseHandle(stdin_rd);
!     CloseHandle(stdout_wr);
!
! #endif /* !UNIX */
!
       return CSCOPE_SUCCESS;
   } /* cs_create_connection */

***************
*** 2097,2104 ****
   /*
    * PRIVATE: cs_release_csp
    *
!  * does the actual free'ing for the cs ptr with an optional flag of whether
!  * or not to free the filename.  called by cs_kill and cs_reset.
    */
       static void
   cs_release_csp(i, freefnpp)
--- 2089,2096 ----
   /*
    * PRIVATE: cs_release_csp
    *
!  * Does the actual free'ing for the cs ptr with an optional flag of whether
!  * or not to free the filename.  Called by cs_kill and cs_reset.
    */
       static void
   cs_release_csp(i, freefnpp)
***************
*** 2116,2125 ****
  	 (void)fputs("q\n", csinfo[i].to_fp);
  	 (void)fflush(csinfo[i].to_fp);
       }
!     /* give cscope chance to exit normally */
!     if (csinfo[i].hProc != NULL
! 	    && WaitForSingleObject(csinfo[i].hProc, 1000) == WAIT_TIMEOUT)
!  TerminateProcess(csinfo[i].hProc, 0);
   #endif

       if (csinfo[i].fr_fp != NULL)
--- 2108,2120 ----
  	 (void)fputs("q\n", csinfo[i].to_fp);
  	 (void)fflush(csinfo[i].to_fp);
       }
!     if (csinfo[i].hProc != NULL)
!     {
!  /* Give cscope a chance to exit normally */
!  if (WaitForSingleObject(csinfo[i].hProc, 1000) == WAIT_TIMEOUT)
! 	    TerminateProcess(csinfo[i].hProc, 0);
!  CloseHandle(csinfo[i].hProc);
!     }
   #endif

       if (csinfo[i].fr_fp != NULL)
***************
*** 2301,2306 ****
--- 2296,2316 ----
       wait_return(TRUE);
       return CSCOPE_SUCCESS;
   } /* cs_show */
+
+
+ /*
+  * PUBLIC: cs_end
+  *
+  * Only called when VIM exits to quit any cscope sessions.
+  */
+     void
+ cs_end()
+ {
+     int i;
+
+     for (i = 0; i < CSCOPE_MAX_CONNECTIONS; i++)
+  cs_release_csp(i, TRUE);
+ }

   #endif /* FEAT_CSCOPE */

*** ../vim-7.1.099/src/if_cscope.h Thu Jun 30 23:59:58 2005
--- src/if_cscope.h Sun Sep  2 16:51:08 2007
***************
*** 72,78 ****
       ino_t     st_ino; /* inode number of cscope db */
   #else
   # if defined(WIN32)
!     int     pid; /* Can't get pid so set it to 0 ;) */
       HANDLE     hProc; /* cscope process handle */
       DWORD     nVolume; /* Volume serial number, instead of st_dev */
       DWORD     nIndexHigh; /* st_ino has no meaning in the Windows */
--- 72,78 ----
       ino_t     st_ino; /* inode number of cscope db */
   #else
   # if defined(WIN32)
!     DWORD     pid; /* PID of the connected cscope process. */
       HANDLE     hProc; /* cscope process handle */
       DWORD     nVolume; /* Volume serial number, instead of st_dev */
       DWORD     nIndexHigh; /* st_ino has no meaning in the Windows */
*** ../vim-7.1.099/src/main.c Thu Aug 30 12:24:21 2007
--- src/main.c Sun Sep  2 16:44:36 2007
***************
*** 1331,1336 ****
--- 1331,1339 ----
   #ifdef FEAT_NETBEANS_INTG
       netbeans_end();
   #endif
+ #ifdef FEAT_CSCOPE
+     cs_end();
+ #endif

       mch_exit(exitval);
   }
***************
*** 3671,3677 ****
  	 mainerr_arg_missing((char_u *)filev[-1]);
       if (mch_dirname(cwd, MAXPATHL) != OK)
  	 return NULL;
!     if ((p = vim_strsave_escaped_ext(cwd, PATH_ESC_CHARS, '\\', TRUE)) ==
NULL)
  	 return NULL;
       ga_init2(&ga, 1, 100);
       ga_concat(&ga, (char_u *)"<C-\\><C-N>:cd ");
--- 3674,3686 ----
  	 mainerr_arg_missing((char_u *)filev[-1]);
       if (mch_dirname(cwd, MAXPATHL) != OK)
  	 return NULL;
!     if ((p = vim_strsave_escaped_ext(cwd,
! #ifdef BACKSLASH_IN_FILENAME
! 		    "",  /* rem_backslash() will tell what chars to escape */
! #else
! 		    PATH_ESC_CHARS,
! #endif
! 		    '\\', TRUE)) == NULL)
  	 return NULL;
       ga_init2(&ga, 1, 100);
       ga_concat(&ga, (char_u *)"<C-\\><C-N>:cd ");
*** ../vim-7.1.099/src/proto/if_cscope.pro Sat May  5 19:15:39 2007
--- src/proto/if_cscope.pro Sun Sep  2 16:51:34 2007
***************
*** 6,9 ****
--- 6,10 ----
   void cs_free_tags __ARGS((void));
   void cs_print_tags __ARGS((void));
   int cs_connection __ARGS((int num, char_u *dbpath, char_u *ppath));
+ void cs_end __ARGS((void));
   /* vim: set ft=c : */
*** ../vim-7.1.099/src/version.c Thu Sep  6 16:33:47 2007
--- src/version.c Thu Sep  6 17:27:51 2007
***************
*** 668,669 ****
--- 668,671 ----
   {   /* Add new patch number below this line */
+ /**/
+     100,
   /**/

--
I have to exercise early in the morning before my brain
figures out what I'm doing.

  /// Bram Moolenaar -- Bram@... -- http://www.Moolenaar.net   \\\
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\        download, build and distribute -- http://www.A-A-P.org        ///
  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

Tony Mechelynck wrote:

> >> At the moment there is still a spurious runtime/plugin/netrw.vim on
> >> the rsync server, which is (IIUC) an obsolete version of
> >> autoload/netrw.vim. Let's hope it will disappear when the above will
> >> appear (presumably autoload/netrw.vim and/or plugin/netrwPlugin.vim
> >> etc.).
> >
> > Oh, that netrw.vim should be in the autoload directory instead of in
> > the plugin directory.  Must be a mistake I made when copying files of a
> > new version.  I'll correct it now.
> >
>
> rsync has just downloaded (among others) autoload/netrw.vim but it is
> v110, not the v112 announced by Dr. Chip at the start of this thread.
> I hope nothing went wrong?

I didn't include v112 yet, pending some questions about the new
"cleanup" command.  If a few people can make remarks on the new version,
such as "works fine for me", then I might include it soon.

--
LAUNCELOT: At last!   A call!  A cry of distress ...
            (he draws his sword, and turns to CONCORDE)
            Concorde!  Brave, Concorde ... you shall not have died in vain!
CONCORDE:  I'm not quite dead, sir ...
                  "Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD

  /// Bram Moolenaar -- Bram@... -- http://www.Moolenaar.net   \\\
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\        download, build and distribute -- http://www.A-A-P.org        ///
  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

On 9/6/07, Christian J. Robinson <infynity@...> wrote:
>
>
> Somewhere between patch 7.1.094 and 7.1.100 I found that setting
> 'lazyredraw' in my .vimrc causes gvim to not fully initialize the
> screen on startup until a key is pressed.  I can reproduce it doing
> this:
>
> echo "set lazyredraw" > vimrc-example
> gvim -u vimrc-example -U NONE --noplugin
>
> Notice that the intro message doesn't get displayed until a key is
> pressed.  If you provide a filename on the command line the buffer
> doesn't get displayed initially either.
>

I have noticed the same problem. Patch 95 is the culprit.

Ajit

--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

Somewhere between patch 7.1.094 and 7.1.100 I found that setting
'lazyredraw' in my .vimrc causes gvim to not fully initialize the
screen on startup until a key is pressed.  I can reproduce it doing
this:

echo "set lazyredraw" > vimrc-example
gvim -u vimrc-example -U NONE --noplugin

Notice that the intro message doesn't get displayed until a key is
pressed.  If you provide a filename on the command line the buffer
doesn't get displayed initially either.

Forcing a redraw on the VimEnter event helps, but the cursor is still
displayed in the wrong location until a key is pressed:

echo "autocmd VimEnter * redraw" >> vimrc-example
gvim -u vimrc-example -U NONE --noplugin

- Christian

--
             Unnamed Law:  If it happens, it must be possible.
Christian J. Robinson <infynity@...> http://infynity.spodzone.com/
    PGP keys: 0x893B0EAF / 0xFB698360   http://infynity.spodzone.com/pgp

--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---