Hi Erik: Looks good to me as well.
/Tim
After looking closer, you are correct. Here is a new webrev: http://cr.openjdk.java.net/~erikj/8055189/webrev.jdk.02/ /Erik On 2014-09-17 15:18, Magnus Ihse Bursie wrote:I think you can split up GensrcCharsetMapping.gmk as well, so it only contains the java.base stuff, and inline the jdk.charsets part directly into Gensrc-jdk.charsets.gmk.
