On 2014-09-17 16:01, Erik Joelsson wrote:
After looking closer, you are correct. Here is a new webrev:http://cr.openjdk.java.net/~erikj/8055189/webrev.jdk.02/
Great! Looks good to me now. Thank you for cleaning this up. /Magnus
/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.
