Your fix looks good. BTW the Webrev doesn’t identify the 3 renamed files, did you use ‘hg rename’?
On 18 Jul 2014, at 18:13, Valerie Peng <valerie.p...@oracle.com> wrote: > Vinnie, > > Could you please help reviewing this change? > > My current approach is to minimize code changes/refactoring - move the > ECParameters, NamedCurve, CurveDB classes to sun.security.util package and > then change the depending classes to use sun.security.util.XXX rather than > sun.security.ec.XXX. > > Besides moving the 3 classes, I also changed some methods to public to be > accessible from sun.security.ec package. > > webrev: http://cr.openjdk.java.net/~valeriep/8035166/webrev.00/ > > Thanks, > Valerie >
