Re: [ptxdist] [PATCH] templates: update vim modeline for makefile and kconfig files

[Bruno Thomsen <bruno.thomsen@gmail.com>]

Den tors. 24. jul. 2025 kl. 09.51 skrev Markus Heidelberg <M.Heidelberg@cab.de>:
>
> On Wed, Jul 23, 2025 at 07:03:42PM +0200, Bruno Thomsen wrote:
> > Den ons. 23. jul. 2025 kl. 09.11 skrev Michael Olbrich
> > <m.olbrich@pengutronix.de>:
> > >
> > > On Wed, Jul 16, 2025 at 09:22:55PM +0200, Bruno Thomsen wrote:
> > > > Setup sane editor default using vim modeline.
> > > >
> > > > sed -i "s/syntax=make/ft=make noet tw=72 ts=8 sw=8/g" rules/templates/*-make
> > >
> > > tw=72 is fine for the .in files. That will give a good flow for the help
> > > text. Most of the rest will be shorter anyways.
> > >
> > > But the makefiles have longer lines. The banner above the stage are already
> > > 78 chars wide and many of the variables at the top as well.
> > > And for the rest, readability really depends  on the content and a defined
> > > textwidth suggests a hard limit that I don't want to set.
> > >
> > > Maybe a soft visual limit with cc=80 instead?
> >
> > [...]
> >
> > Maybe drop tw on both and use cc=80 on kconfig (as you suggested)
> > and cc=100 on makefiles?
>
> I don't think such a visual setting should be imposed on the user and
> could annoy some, myself included ;)
>
> Is this a problem in practice that has to be tackled at all?

Indentation and file format are my primary reasons for adding modeline.
Make it much easier to work on when you switch between 5-10 formats
each day.

Personally I like to have automation tell me when lines need to be broken,
as it can generally give you a hit if you should refactor a code section for
smaller indentation so it's easier to read.

/Bruno

>
> Markus