RFR(M): 8038498: Fix includes and C inlining after 8035330
Thomas Schatzl
thomas.schatzl at oracle.com
Wed Apr 16 19:04:12 UTC 2014
Hi,
On Wed, 2014-04-16 at 18:38 +0200, Volker Simonis wrote:
> Hi Thomas,
>
> thanks for sponsoring this change.
>
> I just wonder why the downported change in jdk8u
> (http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/bfdf528be8e8) is
> attributed to you and not to Goetz?
> In the jdk9 repositories
> (http://hg.openjdk.java.net/jdk9/hs-gc/hotspot/rev/4c16a27793eb) Goetz
> is correctly mentioned as the author.
that's my fault. When committing the backport a few days later, I
forgot to make Goetz the author (i.e. did not use "hg commit -u goetz",
or did not do a straight import of that patch - I do not remember). I
never intended to claim authorship for the change/commit.
There were no changes in the backport, just a straight re-commit. So
there was no review necessary. From the bugtracker (and this mail
thread) it should be clear (at least now) that this is "only" a
backport, and the reason for this difference human error.
Unfortunately the commit message cannot be changed any more.
Interestingly the push hook did not complain that the author is also
mentioned as reviewer :) (Then again, Stefan K is sufficient to pass).
I will try to be more wary of this next time. I hope this poses no
problem.
Sorry,
Thomas
More information about the hotspot-gc-dev
mailing list