diff options
author | Paul Phillips <paulp@improving.org> | 2010-12-20 06:38:16 +0000 |
---|---|---|
committer | Paul Phillips <paulp@improving.org> | 2010-12-20 06:38:16 +0000 |
commit | f40a20b0f418209715529ca5b373f641a5011e0b (patch) | |
tree | 64e8462fc2d0ddb6fc4192f64e73a628d8a4ae3e /src/compiler/scala/tools/nsc/backend/icode/BasicBlocks.scala | |
parent | 0e306e1f90f51efa2d423fd7df810f7cc8dfa915 (diff) | |
download | scala-f40a20b0f418209715529ca5b373f641a5011e0b.tar.gz scala-f40a20b0f418209715529ca5b373f641a5011e0b.tar.bz2 scala-f40a20b0f418209715529ca5b373f641a5011e0b.zip |
I'm wandering around trunk looking for slowness.
are constant distractions which I've meant forever to fix anyway, such
as the importing of collection.mutable._ (or any other package with lots
of reused names.) Why is this relevant to performance? Well,
var x = new HashSet[Int]
What's that? What does 'x += 1' mean? These are questions we can all
live without.
There's almost nothing left which references HashSet or HashMap or Stack
or other ambiguous classes without a qualifier: I can finish trunk but
I can't keep it clean on my own. (Where should I be writing this stuff
down, I wonder.) No review.
Diffstat (limited to 'src/compiler/scala/tools/nsc/backend/icode/BasicBlocks.scala')
-rw-r--r-- | src/compiler/scala/tools/nsc/backend/icode/BasicBlocks.scala | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/compiler/scala/tools/nsc/backend/icode/BasicBlocks.scala b/src/compiler/scala/tools/nsc/backend/icode/BasicBlocks.scala index c7cdad7c20..dbc8c106f3 100644 --- a/src/compiler/scala/tools/nsc/backend/icode/BasicBlocks.scala +++ b/src/compiler/scala/tools/nsc/backend/icode/BasicBlocks.scala @@ -303,7 +303,7 @@ trait BasicBlocks { /** Close the block */ def close { assert(!closed || ignore) - assert(instructionList.length > 0, "Empty block.") + assert(instructionList.nonEmpty, "Empty block.") closed = true setFlag(DIRTYSUCCS) instructionList = instructionList.reverse |