Simplify PatIdent to contain an Ident rather than a Path

Rationale: for what appear to be historical reasons only, the PatIdent contains
a Path rather than an Ident.  This means that there are many places in the code
where an ident is artificially promoted to a path, and---much more problematically---
a bunch of elements from a path are simply thrown away, which seems like an invitation
to some really nasty bugs.

This commit replaces the Path in a PatIdent with a SpannedIdent, which just contains an ident
and a span.
DONT LET THIS COMMIT GO UPSTREAM
simplify and uncomment lambda-var hygiene unit test
comments only
comments only
WIP hygiene for itemfns
remove vestigial parsing of 1 .. id