scala/scala — pull request #2147 by JamesIry (2013-02-20 23:34:28+00:00)
Title: SI-7015 Removes redundant aconst_null; pop; aconst_null creation

For no reason that my code spelunking skills could discern, when a null
constant was used we were emitting the following bytecode

aconst_null
pop
aconst_null

This commit fixes that and includes a test to show that the resulting
byte code still works even without repeating ourselves and/or repeating
ourselves.

Along the way I found some other questionable things in adapt and I've
added TODO's and issue https://issues.scala-lang.org/browse/SI-7159 to
track.