If you declare a member class that does not require access to an enclosing instance, always put the static modifier in its declaration,
A more actionable way to put this suggestion: Consider starting with the static modifier in place, and only remove it when it has become apparent that the class is better off as an inner class.