diff options
author | Christopher Vogt <oss.nsp@cvogt.org> | 2016-04-27 09:51:14 -0400 |
---|---|---|
committer | Christopher Vogt <oss.nsp@cvogt.org> | 2016-04-28 13:34:01 -0400 |
commit | 1de728fc751669fb08b5263d5ba4df11b64afda0 (patch) | |
tree | e362a2c719107c49ed634235ec9f40e22bcc3461 /stage1/CachingClassLoader.scala | |
parent | b37ccc12d3d6e8fc5fcf2730fc498f5bb8b08655 (diff) | |
download | cbt-1de728fc751669fb08b5263d5ba4df11b64afda0.tar.gz cbt-1de728fc751669fb08b5263d5ba4df11b64afda0.tar.bz2 cbt-1de728fc751669fb08b5263d5ba4df11b64afda0.zip |
for better performance do not generate exceptions during class loading for control flow, but use null instead
Diffstat (limited to 'stage1/CachingClassLoader.scala')
-rw-r--r-- | stage1/CachingClassLoader.scala | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/stage1/CachingClassLoader.scala b/stage1/CachingClassLoader.scala index e75f14c..4ddebda 100644 --- a/stage1/CachingClassLoader.scala +++ b/stage1/CachingClassLoader.scala @@ -5,8 +5,13 @@ import scala.util.Try trait CachingClassLoader extends ClassLoader{ def logger: Logger - val cache = new KeyLockedLazyCache[String,Try[Class[_]]]( new ConcurrentHashMap, new ConcurrentHashMap, Some(logger) ) + val cache = new KeyLockedLazyCache[String,Option[Class[_]]]( new ConcurrentHashMap, new ConcurrentHashMap, Some(logger) ) override def loadClass(name: String, resolve: Boolean) = { - cache.get( name, Try(super.loadClass(name, resolve)) ).get + cache.get( name, Try(super.loadClass(name, resolve)).toOption ).getOrElse(null) + } + override def loadClass(name: String) = { + val _class = super.loadClass(name) + if(_class == null) throw new ClassNotFoundException(name) + else _class } } |