Making these protected, public (inherited from class) is too much. These nested classes should only be instantiated from within their parent class.
Some people are overly sensitive to method chaining, a practice that's becoming more and more popular these days.
Bloating the changelist with variable renaming.
Reverting this; since the contents are modified inside the loop, the behavior is undefined if we iterate over the entrySet in the traditional way. Still looks ugly AF.