On Tuesday 30 October 2012 13:40:22 Sebastian Kügler wrote:
> On Tuesday, October 30, 2012 11:35:50 Aaron J. Seigo wrote:
> > > >> 5. Vim:
> > > >>> The {{{ and }}} are just for the sakes of vim's automatic folding.
> > > >>
> > > >> Do we all use vim? I think no. So, we must never force developers use
> > > >> vim.
> > > >
> > > > i mostly agree :) i don't think this forces anyone to use vim, but i
> > > > would
> > > > like to see a solution that works with katepart before adding it to
> > > > the
> > > > style guide.
> > >
> > > Do you want force developers use kate?
> >
> > of course not :)
> >
> > but i'd like to make the lives of people who use kate, vim, etc. easier if
> > possible. if we have to chose between "no one benefits from code folding"
> > or "some people get to benefit", we should pick "some people" every
> > single
> > time. "some" is better than "none".
> >
> > this does not prevent you from using any editor you wish -> you might not
> > get the cute little bonus of code folding, but otherwise it's the same as
> > it is right now already.
>
> To be honest, I don't like that. I'm fine with people putting it in, but
> making it mandatory is not OK with me. I would not like to reject someone's
> patch based on "its properties aren't foldable in vim". First, this is 2013.
> Then, property definitions aren't usually pages long, I've rarely seen more
> than 10.
>
> -1 one if it was the idea to mandate {{{ annd }}} in comments.
>
> (Allowing it is fine with me, mandating is is not.)
Personally I'm even against allowing it go in at all. It just clutters the
code. It's not our task to cover up bugs in editors. If vim doesn't handle it,
vim should be fixed. Also given that I'm not a vim user I would not know how
many }}}}}} I have to place at a position or whether I have to change them
when moving code around.Personally I also are against the Kate settings you find in some files. If someone needs that, that's fine with me, but it should not end up in the repository. Cheers Martin
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
