> On Jul 12, 2017, at 8:20 AM, Alex Harui <aha...@adobe.com.INVALID> wrote: > > Again, though, I think this optimization isn't urgent.
I completely agree. That’s why I have not been bringing this up despite it being on my mind. When the discussion came up, I couldn’t help but join. ;-) > goog.DEBUG is already being used in Language.as. Thanks! I hadn’t noticed. I was missing an import of of goog.DEBUG in COMPILE::JS I’m guessing the imports of goog.bind and goog.global was enough to make goog.DEBUG visible to the compiler in Language.as. Once we’re on this topic, there’s something that I had wanted to bring up for a long time: I think trace statements should disappear in the release JS build. Should we put all the JS trace code inside an if(goog.DEBUG) block? Thanks, Harbs
