home *** CD-ROM | disk | FTP | other *** search
/ vim.ftp.fu-berlin.de / 2015-02-03.vim.ftp.fu-berlin.de.tar / vim.ftp.fu-berlin.de / patches / 7.3 / 7.3.380 < prev    next >
Encoding:
Internet Message Format  |  2012-11-20  |  7.6 KB
  1. To: vim_dev@googlegroups.com
  2. Subject: Patch 7.3.380
  3. Fcc: outbox
  4. From: Bram Moolenaar <Bram@moolenaar.net>
  5. Mime-Version: 1.0
  6. Content-Type: text/plain; charset=UTF-8
  7. Content-Transfer-Encoding: 8bit
  8. ------------
  9.  
  10. Patch 7.3.380
  11. Problem:    C-indenting wrong for a function header.
  12. Solution:   Skip to the start paren. (Lech Lorens)
  13. Files:        src/misc1.c, src/testdir/test3.in, src/testdir/test3.ok
  14.  
  15.  
  16. *** ../vim-7.3.379/src/misc1.c    2011-12-14 20:05:17.000000000 +0100
  17. --- src/misc1.c    2011-12-14 20:16:43.000000000 +0100
  18. ***************
  19. *** 4943,4949 ****
  20.   static int    cin_islinecomment __ARGS((char_u *));
  21.   static int    cin_isterminated __ARGS((char_u *, int, int));
  22.   static int    cin_isinit __ARGS((void));
  23. ! static int    cin_isfuncdecl __ARGS((char_u **, linenr_T));
  24.   static int    cin_isif __ARGS((char_u *));
  25.   static int    cin_iselse __ARGS((char_u *));
  26.   static int    cin_isdo __ARGS((char_u *));
  27. --- 4943,4949 ----
  28.   static int    cin_islinecomment __ARGS((char_u *));
  29.   static int    cin_isterminated __ARGS((char_u *, int, int));
  30.   static int    cin_isinit __ARGS((void));
  31. ! static int    cin_isfuncdecl __ARGS((char_u **, linenr_T, linenr_T, int, int));
  32.   static int    cin_isif __ARGS((char_u *));
  33.   static int    cin_iselse __ARGS((char_u *));
  34.   static int    cin_isdo __ARGS((char_u *));
  35. ***************
  36. *** 5585,5605 ****
  37.    * "sp" points to a string with the line.  When looking at other lines it must
  38.    * be restored to the line.  When it's NULL fetch lines here.
  39.    * "lnum" is where we start looking.
  40.    */
  41.       static int
  42. ! cin_isfuncdecl(sp, first_lnum)
  43.       char_u    **sp;
  44.       linenr_T    first_lnum;
  45.   {
  46.       char_u    *s;
  47.       linenr_T    lnum = first_lnum;
  48.       int        retval = FALSE;
  49.   
  50.       if (sp == NULL)
  51.       s = ml_get(lnum);
  52.       else
  53.       s = *sp;
  54.   
  55.       /* Ignore line starting with #. */
  56.       if (cin_ispreproc(s))
  57.       return FALSE;
  58. --- 5585,5621 ----
  59.    * "sp" points to a string with the line.  When looking at other lines it must
  60.    * be restored to the line.  When it's NULL fetch lines here.
  61.    * "lnum" is where we start looking.
  62. +  * "min_lnum" is the line before which we will not be looking.
  63.    */
  64.       static int
  65. ! cin_isfuncdecl(sp, first_lnum, min_lnum, ind_maxparen, ind_maxcomment)
  66.       char_u    **sp;
  67.       linenr_T    first_lnum;
  68. +     linenr_T    min_lnum;
  69. +     int        ind_maxparen;
  70. +     int        ind_maxcomment;
  71.   {
  72.       char_u    *s;
  73.       linenr_T    lnum = first_lnum;
  74.       int        retval = FALSE;
  75. +     pos_T    *trypos;
  76. +     int        just_started = TRUE;
  77.   
  78.       if (sp == NULL)
  79.       s = ml_get(lnum);
  80.       else
  81.       s = *sp;
  82.   
  83. +     if (find_last_paren(s, '(', ')')
  84. +     && (trypos = find_match_paren(ind_maxparen, ind_maxcomment)) != NULL)
  85. +     {
  86. +     lnum = trypos->lnum;
  87. +     if (lnum < min_lnum)
  88. +         return FALSE;
  89. + 
  90. +     s = ml_get(lnum);
  91. +     }
  92. + 
  93.       /* Ignore line starting with #. */
  94.       if (cin_ispreproc(s))
  95.       return FALSE;
  96. ***************
  97. *** 5650,5662 ****
  98.           /* Require a comma at end of the line or a comma or ')' at the
  99.            * start of next line. */
  100.           s = skipwhite(s);
  101. !         if (!comma && *s != ',' && *s != ')')
  102.           break;
  103.       }
  104.       else if (cin_iscomment(s))    /* ignore comments */
  105.           s = cin_skipcomment(s);
  106.       else
  107.           ++s;
  108.       }
  109.   
  110.   done:
  111. --- 5666,5682 ----
  112.           /* Require a comma at end of the line or a comma or ')' at the
  113.            * start of next line. */
  114.           s = skipwhite(s);
  115. !         if (!just_started && (!comma && *s != ',' && *s != ')'))
  116.           break;
  117. +         just_started = FALSE;
  118.       }
  119.       else if (cin_iscomment(s))    /* ignore comments */
  120.           s = cin_skipcomment(s);
  121.       else
  122. +     {
  123.           ++s;
  124. +         just_started = FALSE;
  125. +     }
  126.       }
  127.   
  128.   done:
  129. ***************
  130. *** 7158,7164 ****
  131.                * (it's a variable declaration).
  132.                */
  133.               if (start_brace != BRACE_IN_COL0
  134. !                 || !cin_isfuncdecl(&l, curwin->w_cursor.lnum))
  135.               {
  136.                   /* if the line is terminated with another ','
  137.                    * it is a continued variable initialization.
  138. --- 7178,7185 ----
  139.                * (it's a variable declaration).
  140.                */
  141.               if (start_brace != BRACE_IN_COL0
  142. !                 || !cin_isfuncdecl(&l, curwin->w_cursor.lnum,
  143. !                          0, ind_maxparen, ind_maxcomment))
  144.               {
  145.                   /* if the line is terminated with another ','
  146.                    * it is a continued variable initialization.
  147. ***************
  148. *** 8019,8025 ****
  149.           && vim_strchr(theline, '}') == NULL
  150.           && !cin_ends_in(theline, (char_u *)":", NULL)
  151.           && !cin_ends_in(theline, (char_u *)",", NULL)
  152. !         && cin_isfuncdecl(NULL, cur_curpos.lnum + 1)
  153.           && !cin_isterminated(theline, FALSE, TRUE))
  154.       {
  155.           amount = ind_func_type;
  156. --- 8040,8048 ----
  157.           && vim_strchr(theline, '}') == NULL
  158.           && !cin_ends_in(theline, (char_u *)":", NULL)
  159.           && !cin_ends_in(theline, (char_u *)",", NULL)
  160. !         && cin_isfuncdecl(NULL, cur_curpos.lnum + 1,
  161. !                   cur_curpos.lnum + 1,
  162. !                   ind_maxparen, ind_maxcomment)
  163.           && !cin_isterminated(theline, FALSE, TRUE))
  164.       {
  165.           amount = ind_func_type;
  166. ***************
  167. *** 8125,8131 ****
  168.            * If the line looks like a function declaration, and we're
  169.            * not in a comment, put it the left margin.
  170.            */
  171. !         if (cin_isfuncdecl(NULL, cur_curpos.lnum))  /* XXX */
  172.               break;
  173.           l = ml_get_curline();
  174.   
  175. --- 8148,8155 ----
  176.            * If the line looks like a function declaration, and we're
  177.            * not in a comment, put it the left margin.
  178.            */
  179. !         if (cin_isfuncdecl(NULL, cur_curpos.lnum, 0,
  180. !                    ind_maxparen, ind_maxcomment))  /* XXX */
  181.               break;
  182.           l = ml_get_curline();
  183.   
  184. ***************
  185. *** 8173,8179 ****
  186.            * line (and the ones that follow) needs to be indented as
  187.            * parameters.
  188.            */
  189. !         if (cin_isfuncdecl(&l, curwin->w_cursor.lnum))
  190.           {
  191.               amount = ind_param;
  192.               break;
  193. --- 8197,8204 ----
  194.            * line (and the ones that follow) needs to be indented as
  195.            * parameters.
  196.            */
  197. !         if (cin_isfuncdecl(&l, curwin->w_cursor.lnum, 0,
  198. !                    ind_maxparen, ind_maxcomment))
  199.           {
  200.               amount = ind_param;
  201.               break;
  202. *** ../vim-7.3.379/src/testdir/test3.in    2011-12-14 20:05:17.000000000 +0100
  203. --- src/testdir/test3.in    2011-12-14 20:11:24.000000000 +0100
  204. ***************
  205. *** 1429,1435 ****
  206.   
  207.   STARTTEST
  208.   :set cino&
  209. ! 2kdd=4][
  210.   ENDTEST
  211.   
  212.   void func(void)
  213. --- 1429,1435 ----
  214.   
  215.   STARTTEST
  216.   :set cino&
  217. ! 2kdd=7][
  218.   ENDTEST
  219.   
  220.   void func(void)
  221. ***************
  222. *** 1478,1484 ****
  223.       3, 4,
  224.       5, 6};
  225.   
  226. ! printf("Don't you dare indent this line incorrectly!\n);
  227.   }
  228.   
  229.   STARTTEST
  230. --- 1478,1506 ----
  231.       3, 4,
  232.       5, 6};
  233.   
  234. ! printf("Don't you dare indent this line incorrectly!\n");
  235. ! }
  236. ! 
  237. ! void
  238. ! func4(a, b,
  239. !         c)
  240. ! int a;
  241. ! int b;
  242. ! int c;
  243. ! {
  244. ! }
  245. ! 
  246. ! void
  247. ! func5(
  248. !         int a,
  249. !         int b)
  250. ! {
  251. ! }
  252. ! 
  253. ! void
  254. ! func6(
  255. !         int a)
  256. ! {
  257.   }
  258.   
  259.   STARTTEST
  260. *** ../vim-7.3.379/src/testdir/test3.ok    2011-12-14 20:05:17.000000000 +0100
  261. --- src/testdir/test3.ok    2011-12-14 20:11:24.000000000 +0100
  262. ***************
  263. *** 1331,1337 ****
  264.           3, 4,
  265.           5, 6};
  266.   
  267. !     printf("Don't you dare indent this line incorrectly!\n);
  268.   }
  269.   
  270.   
  271. --- 1331,1359 ----
  272.           3, 4,
  273.           5, 6};
  274.   
  275. !     printf("Don't you dare indent this line incorrectly!\n");
  276. ! }
  277. ! 
  278. !     void
  279. ! func4(a, b,
  280. !         c)
  281. !     int a;
  282. !     int b;
  283. !     int c;
  284. ! {
  285. ! }
  286. ! 
  287. !     void
  288. ! func5(
  289. !         int a,
  290. !         int b)
  291. ! {
  292. ! }
  293. ! 
  294. !     void
  295. ! func6(
  296. !         int a)
  297. ! {
  298.   }
  299.   
  300.   
  301. *** ../vim-7.3.379/src/version.c    2011-12-14 20:05:17.000000000 +0100
  302. --- src/version.c    2011-12-14 20:20:50.000000000 +0100
  303. ***************
  304. *** 716,717 ****
  305. --- 716,719 ----
  306.   {   /* Add new patch number below this line */
  307. + /**/
  308. +     380,
  309.   /**/
  310.  
  311. -- 
  312. "Intelligence has much less practical application than you'd think."
  313.           -- Scott Adams, Dilbert.
  314.  
  315.  /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
  316. ///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
  317. \\\  an exciting new programming language -- http://www.Zimbu.org        ///
  318.  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///
  319.