> So we have people who prefer 80 columns, and people who prefer unlimited
columns.  There's not really a compromise position between these two
options, so how do we choose?

We have people who prefer 80 columns and we have people who prefer more.
I prefer 80 columns.
So there is no column issue issue to add a .clang-format file to the Mixxx
source and ask any contributor to run it if there are style issues.

The latest discussion was about the code cluttering produced by a
unsupervised run at all Mixxx source.
This happens because we allow currently allow more than 80 columns for
reasons.

Setting the columns to unlimited helps a bit.
But there are still other cluttering issues that requires developer
attention.


> Never seen astyle has broke something like code didn't work anymore.

Do you have experience with Astyle. Maybe this can be configured to be more
temperate than clang-format.
It would be also nice if we can do the mass auto-format for single style
issues only e.g. the pinter * alignment.






2015-06-11 15:30 GMT+02:00 Owen Williams <owilli...@mixxx.org>:

> So we have people who prefer 80 columns, and people who prefer unlimited
> columns.  There's not really a compromise position between these two
> options, so how do we choose?
>
>
> On Wed, 2015-06-10 at 09:28 -0400, Owen Williams wrote:
> > On Wed, 2015-06-10 at 09:16 -0400, RJ Ryan wrote:
> > >
> >
> > >
> > >
> > > The part of clang-format which frees you from having to talk about
> > > code style is that there is only one right way to format the code.
> > > Letting the developer have any choice in the matter goes against this.
> > >
> >
> > Agree.  I don't really feel strongly about line length, so 80 is fine
> > with me.
> >
> >
> >
> >
> ------------------------------------------------------------------------------
> > _______________________________________________
> > Get Mixxx, the #1 Free MP3 DJ Mixing software Today
> > http://mixxx.org
> >
> >
> > Mixxx-devel mailing list
> > Mixxx-devel@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/mixxx-devel
> >
> >
>
>
>
>
> ------------------------------------------------------------------------------
> _______________________________________________
> Get Mixxx, the #1 Free MP3 DJ Mixing software Today
> http://mixxx.org
>
>
> Mixxx-devel mailing list
> Mixxx-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/mixxx-devel
>
------------------------------------------------------------------------------
_______________________________________________
Get Mixxx, the #1 Free MP3 DJ Mixing software Today
http://mixxx.org


Mixxx-devel mailing list
Mixxx-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mixxx-devel

Reply via email to