68574Re: [PATCH] add 'cinoption' "E" to correctly indent enum for Java code
- Mar 5 11:57 AMHong Xu wrote:
> On Tuesday, March 5, 2013 8:16:48 AM UTC-8, Lech Lorens wrote:Thanks, tests are good. However, it seems the test passes without your
> > Wouldn't it be better to use the existing 'cinoptions' entry jN? It's
> > related to Java...
> > BTW, in the past I made a number of modifications to the indenting
> > code. It was like treading on a mine field (lots of things I could
> > break). Would you, please, also include a test case for your
> > functionality so that the next person that modifies get_c_indent()
> > does not break what you've achieved?
> Thanks for the reminder. I have included the test case in the new
> attached patch.
"static" is already checked for and skipped above the lines you insert.
Without a flag to enable this. I don't see much of a problem also
skipping the public/private/protected attributes without a flag.
Actually, what really matters is the check for "=". I think it should
be ignored for Java, the "j" flag.
How To Keep A Healthy Level Of Insanity:
11. Specify that your drive-through order is "to go".
/// Bram Moolenaar -- Bram@... -- http://www.Moolenaar.net \\\
/// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\ an exciting new programming language -- http://www.Zimbu.org ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
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 because you are subscribed to the Google Groups "vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email to vim_dev+unsubscribe@....
For more options, visit https://groups.google.com/groups/opt_out.
- << Previous post in topic Next post in topic >>