[lworld] Integrated: 8261393: [lworld] Adopt `primitive' as the modifier in declaration of identity free classes in lieu of `inline`

Srikanth Adayapalam srikanth.adayapalam at oracle.com
Tue Feb 9 12:02:09 UTC 2021


Hello!

Please take note of this push. It is not inline class anymore, but primitive class as in


primitive class Point implements Shape {
    private double x;
    private double y;

    public Point(double x, double y) {
        this.x = x;
        this.y = y;
    }

    public double x() { return x; }
    public double y() { return y; }

    public Point translate(double dx, double dy) {
        return new Point(x+dx, y+dy);
    }

    public boolean contains(Point p) {
        return equals(p);
    }
}


I have adjusted all the tests across components. This change merely adjusts the declaration sites.
It is likely that comments in source and tests, APIs internal and external, identifiers, documentation,
various experimental command line options etc still refer to inline classes. Respective component owners
should follow up.

Javac messages are not adjusted yet and will refer to inline types. I will follow up.

Srikanth
________________________________
From: valhalla-dev <valhalla-dev-retn at openjdk.java.net> on behalf of Srikanth Adayapalam <sadayapalam at openjdk.java.net>
Sent: 09 February 2021 17:17
To: valhalla-dev at openjdk.java.net <valhalla-dev at openjdk.java.net>
Subject: [lworld] Integrated: 8261393: [lworld] Adopt `primitive' as the modifier in declaration of identity free classes in lieu of `inline`

On Tue, 9 Feb 2021 11:35:41 GMT, Srikanth Adayapalam <sadayapalam at openjdk.org> wrote:

> Compiler changes to accept primitive as the modifier in class declaration plus test changes across components necessitated by said change.

This pull request has now been integrated.

Changeset: 693282f3
Author:    Srikanth Adayapalam <sadayapalam at openjdk.org>
URL:       https://git.openjdk.java.net/valhalla/commit/693282f3
Stats:     678 lines in 275 files changed: 40 ins; 41 del; 597 mod

8261393: [lworld] Adopt `primitive' as the modifier in declaration of identity free classes in lieu of `inline`

-------------

PR: https://git.openjdk.java.net/valhalla/pull/328


More information about the valhalla-dev mailing list