Description
More specifically, the MarkupContainer's add(final Component child) calls checkHierarchyChange() before determining if the child reference is null. IMHO this should be ideally be called after checking if the child reference is null. From a user perspective, throwing an IllegalArgumentException with a message telling them that they have passed a null referenced component has better affordance then an esoteric NNE on calling checkHierarchyChange.