On 9/19/18 7:46 PM, Jaikiran Pai wrote:
Thank you. More of a FYI and if it matters from a process point of view
- in a couple of my earlier contributions, the sponsors have used the
"Contributed-by" line to be "Jaikiran Pai <jaikiran....@gmail.com>" like
here http://hg.openjdk.java.net/jdk/jdk/rev/6c394ed56b07. I don't have
any specific preference on which one is used.

I'm happy to change the Contributed-by line to anything you want. After all this represents YOU and it's recorded in the changeset history. It's probably good to be consistent. Unfortunately I see the following changesets in the history right now:

changeset:   51755:6c394ed56b07
user:        xuelei
date:        Fri Sep 14 20:30:28 2018 -0700
files:       
src/java.base/share/classes/javax/net/ssl/X509ExtendedKeyManager.java
description:
8210785: Trivial typo fix in X509ExtendedKeyManager javadoc
Reviewed-by: xuelei
Contributed-by: Jaikiran Pai <jaikiran....@gmail.com>


changeset:   49926:b7c2996d690b
user:        chegar
date:        Mon Apr 30 16:13:30 2018 +0100
files: src/java.base/share/classes/java/net/InetAddress.java test/jdk/java/net/InetAddress/GetLoopbackAddress.java
description:
8201545: InetAddress.getByName/getAllByName should clarify empty String behavior
Reviewed-by: chegar
Contributed-by: Jaikiran Pai <jai.forums2...@gmail.com>


Since you mentioned it I'll proceed with "Jaikiran Pai <jaikiran....@gmail.com>" since that looks a bit more "official". But do let me know if you have a different preference.

I re-read the whole javadoc contained in your modified patch and it all
looks good to me. Thank you very much for the help in sponsoring,
reviewing and providing detailed inputs during the review.

You're welcome. OK, I'll proceed with the CSR.

s'marks

Reply via email to