6.2.282

   1 To: vim-dev@vim.org
   2 Subject: Patch 6.2.282
   3 Fcc: outbox
   4 From: Bram Moolenaar <Bram@moolenaar.net>
   5 Mime-Version: 1.0
   6 Content-Type: text/plain; charset=ISO-8859-1
   7 Content-Transfer-Encoding: 8bit
   8 ------------
   9
  10 Patch 6.2.282
  11 Problem:    When using CTRL-O to go back to a help file, it becomes listed.
  12             (Andrew Nesbit)
  13             Using ":tag" or ":tjump" in a help file doesn't keep the help file
  14             settings (e.g. for 'iskeyword').
  15 Solution:   Don't mark a buffer as listed when its help flag is set.  Put all
  16             the option settings for a help buffer together in do_ecmd().
  17 Files:      src/ex_cmds.c
  18
  19
  20 *** ../vim-6.2.281/src/ex_cmds.c        Wed Feb  4 21:42:30 2004
  21 --- src/ex_cmds.c       Fri Feb 20 22:00:27 2004
  22 ***************
  23 *** 2835,2846 ****
  24
  25       if ((flags & ECMD_SET_HELP) || keep_help_flag)
  26       {
  27         curbuf->b_help = TRUE;
  28   #ifdef FEAT_QUICKFIX
  29         set_string_option_direct((char_u *)"buftype", -1,
  30 !                                      (char_u *)"help", OPT_FREE |OPT_LOCAL);
  31   #endif
  32 !       curbuf->b_p_ma = FALSE;
  33         curbuf->b_p_bin = FALSE;        /* reset 'bin' before reading file */
  34         curwin->w_p_nu = 0;             /* no line numbers */
  35   #ifdef FEAT_SCROLLBIND
  36 --- 2835,2873 ----
  37
  38       if ((flags & ECMD_SET_HELP) || keep_help_flag)
  39       {
  40 +       char_u  *p;
  41 +
  42         curbuf->b_help = TRUE;
  43   #ifdef FEAT_QUICKFIX
  44         set_string_option_direct((char_u *)"buftype", -1,
  45 !                                      (char_u *)"help", OPT_FREE|OPT_LOCAL);
  46 ! #endif
  47 !
  48 !       /*
  49 !        * Always set these options after jumping to a help tag, because the
  50 !        * user may have an autocommand that gets in the way.
  51 !        * Accept all ASCII chars for keywords, except ' ', '*', '"', '|', and
  52 !        * latin1 word characters (for translated help files).
  53 !        * Only set it when needed, buf_init_chartab() is some work.
  54 !        */
  55 !       p =
  56 ! #ifdef EBCDIC
  57 !               (char_u *)"65-255,^*,^|,^\"";
  58 ! #else
  59 !               (char_u *)"!-~,^*,^|,^\",192-255";
  60   #endif
  61 !       if (STRCMP(curbuf->b_p_isk, p) != 0)
  62 !       {
  63 !           set_string_option_direct((char_u *)"isk", -1, p,
  64 !                                                         OPT_FREE|OPT_LOCAL);
  65 !           check_buf_options(curbuf);
  66 !           (void)buf_init_chartab(curbuf, FALSE);
  67 !       }
  68 !
  69 !       curbuf->b_p_ts = 8;             /* 'tabstop' is 8 */
  70 !       curwin->w_p_list = FALSE;       /* no list mode */
  71 !
  72 !       curbuf->b_p_ma = FALSE;         /* not modifiable */
  73         curbuf->b_p_bin = FALSE;        /* reset 'bin' before reading file */
  74         curwin->w_p_nu = 0;             /* no line numbers */
  75   #ifdef FEAT_SCROLLBIND
  76 ***************
  77 *** 2869,2875 ****
  78   #ifdef FEAT_AUTOCMD
  79         buf = curbuf;
  80   #endif
  81 !       set_buflisted(TRUE);
  82       }
  83
  84   #ifdef FEAT_AUTOCMD
  85 --- 2896,2905 ----
  86   #ifdef FEAT_AUTOCMD
  87         buf = curbuf;
  88   #endif
  89 !       /* Don't make a buffer listed if it's a help buffer.  Useful when
  90 !        * using CTRL-O to go back to a help file. */
  91 !       if (!curbuf->b_help)
  92 !           set_buflisted(TRUE);
  93       }
  94
  95   #ifdef FEAT_AUTOCMD
  96 ***************
  97 *** 4649,4677 ****
  98       if (alt_fnum != 0 && curwin->w_alt_fnum == empty_fnum)
  99         curwin->w_alt_fnum = alt_fnum;
 100
 101 -     /*
 102 -      * Always set these options after jumping to a help tag, because the user
 103 -      * may have an autocommand that gets in the way.
 104 -      * Accept all ASCII chars for keywords, except ' ', '*', '"', '|', and
 105 -      * latin1 word characters (for translated help files).
 106 -      * Only set it when needed, buf_init_chartab() is some work.
 107 -      */
 108 -     p =
 109 - #ifdef EBCDIC
 110 -           (char_u *)"65-255,^*,^|,^\"";
 111 - #else
 112 -           (char_u *)"!-~,^*,^|,^\",192-255";
 113 - #endif
 114 -     if (STRCMP(curbuf->b_p_isk, p) != 0)
 115 -     {
 116 -       set_string_option_direct((char_u *)"isk", -1, p, OPT_FREE|OPT_LOCAL);
 117 -       check_buf_options(curbuf);
 118 -       (void)buf_init_chartab(curbuf, FALSE);
 119 -     }
 120 -
 121 -     curbuf->b_p_ts = 8;
 122 -     curwin->w_p_list = FALSE;
 123 -
 124   erret:
 125       if (need_free)
 126         vim_free(arg);
 127 --- 4705,4710 ----
 128 *** ../vim-6.2.281/src/version.c        Fri Feb 20 21:53:42 2004
 129 --- src/version.c       Fri Feb 20 21:54:54 2004
 130 ***************
 131 *** 639,640 ****
 132 --- 639,642 ----
 133   {   /* Add new patch number below this line */
 134 + /**/
 135 +     282,
 136   /**/
 137
 138 --
 139 hundred-and-one symptoms of being an internet addict:
 140 205. You're constantly yelling at your spouse, family, roommate, whatever,
 141      for using the phone for stupid things...like talking.
 142
 143  /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
 144 ///        Sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
 145 \\\              Project leader for A-A-P -- http://www.A-A-P.org        ///
 146  \\\  Help AIDS victims, buy here: http://ICCF-Holland.org/click1.html  ///