#1211 com.sun packages

vkuzkokov Tue 14 Sep 2010

$FAN_HOME/src/compilerJava/fan/ClassPath.fan from line 140 reads:

private Void accept(Str:Str[] acc, Uri uri)
{
  if (uri.ext != "class") return
  package := uri.path[0..-2].join(".")
  if (package.startsWith("com.sun") || package.startsWith("sun")) return
  name := uri.basename
  if (name == "Void") return
  classes := acc[package]
  if (classes == null) acc[package] = classes = Str[,]
  if (!classes.contains(name)) classes.add(name)
}

While specs explicitly say than sun packages are dangerous, com.sun may contain something useful like com.sun.mail.smtp or com.sun.javadoc.

brian Tue 14 Sep 2010

That is mostly just an optimization since the process we use to figure out packages is so hacked. I would say the safest thing is to leave that behavior for now. Then once (if ever) Jigsaw comes out, that whole design is going to have to be redone.

Login or Signup to reply.