Given that `drawStem` takes no parameters and is called just once, I think it doesn't need to be in an inner function. If the intent here is to break up the code into more modular chunks, then perhaps move the logic to the prototype `drawStem`?