Review request for JMC-6061: running cleanup on core
Mario Torre
neugens at redhat.com
Tue Jul 3 13:36:52 UTC 2018
On Tue, Jul 3, 2018 at 2:51 PM, Marcus Hirt <marcus.hirt at oracle.com> wrote:
> Hi all,
>
> This patch is not quite as scary as it looks. Many files have whitespace
> changes only. That said, I'd rather do these changes as early as possible.
>
> Jira: https://bugs.openjdk.java.net/browse/JMC-6061
> Webrev: http://cr.openjdk.java.net/~hirt/JMC-6061/webrev.0/
>
> Kind regards,
> Marcus
Hi Marcus,
I pick an easy one for the start :)
The patch looks generally good (well, as you say is mostly about white
spaces), I have a question though, do we have a coding guidelines?
For instance, we are aligning now the enum fields to the same column
and the start of the definition, is that the recommended style?
Cheers,
Mario
--
Mario Torre
Associate Manager, Software Engineering
Red Hat GmbH <https://www.redhat.com>
9704 A60C B4BE A8B8 0F30 9205 5D7E 4952 3F65 7898
More information about the jmc-dev
mailing list