[icedtea-web] Proposed formatting for C++ side of plugin
Deepak Bhole
dbhole at redhat.com
Tue May 22 13:49:44 PDT 2012
* Deepak Bhole <dbhole at redhat.com> [2012-05-22 16:48]:
> * Adam Domurad <adomurad at redhat.com> [2012-05-22 16:06]:
> > So, currently the C++ side of the plugin uses inconsistent indents (tabs
> > and various # of spaces). An auto-format to make the code formatted more
> > like the Java side could be beneficial.
> >
> > As Omair pointed out however, formatting HEAD will cause back-ports to
> > become more difficult.
> >
> > I therefore propose to create an Eclipse formatting profile that will
> > define the standard formatting for the C++ side of the project, and then
> > back-port this formatting profile so that future back-ported patches are
> > not made more difficult due to indenting/line number differences.
> >
>
> Yep, there is a formatting profile for Java in the repos -- we'd just
> have to add something similar for C++.
>
> Also, I am unclear as to what you mean by "back-port this formatting
> profile" -- as in re-indent the code in stable branches?
>
Adam just confirmed that this indeed entails backporting the indentation
changes.
I am OK with this as it will strictly be a spacing only change and no
matter when we do it, we will have to backport it to make future
backporting easier.
Given the scope, I think at least one more person should sign off on
this one however.
Cheers,
Deepak
More information about the distro-pkg-dev
mailing list