Github user HeartSaVioR commented on the issue:
https://github.com/apache/storm/pull/2438
So there's `hasNext()` in `waitingToEmit()` which made me confused...
I agree technically the patch may not be required, but I'd like to propose
removing `hasNext()` from `waitingToEmit()` and adopt this patch, which helps
to read `emit()` method.
And I prefer throwing out the list instead of removing elements if we
discard the list just after loop end, but this is not the case so I'm +1 to
remove element when element is not needed.
---
