Hello Serhiy,

I've not reviewed the patch but I trust that if you say it simplifies things, that's so.  Perhaps the important question is if it is okay to change the AST in backwards incompatible ways within 3.x releases.  As a library author who could be affected, I think it is okay. The AST is mostly an implementation detail and should not required to maintain compatibility.  We expose it via the 'ast' module for low level tools that want to manipulate it.  It is up to those users to handle backwards incompatible changes.

That said, we shouldn't just change it for trivial reasons.  That just causes work for 3rd party libraries.  Removing 400 lines of code seems like sufficient reason.