reorder clauses in item parser
rename parse_non_foreign_item_or_view_item to parse_item_or_view_item
remove jbcdones, should cancel before upstream
undo abstraction over whether to parse attrs in a block

In principle, it seems like a nice idea to abstract over the two
functions that parse blocks (one with inner attrs allowed, one not).
However, the existing one wound up making things more complex than
just having two separate functions, especially after the obsolete
syntax is (will be) removed.
parser comments