]> git.lizzy.rs Git - rust.git/commit
Overhaul `MacArgs::Eq`.
authorNicholas Nethercote <n.nethercote@gmail.com>
Thu, 28 Apr 2022 20:52:01 +0000 (06:52 +1000)
committerNicholas Nethercote <n.nethercote@gmail.com>
Wed, 4 May 2022 21:06:12 +0000 (07:06 +1000)
commitc318cf453dafbd8129203c3117d8f7be799f62ef
tree0d0cc61c5b463a37a96fe905001001ed9a677b0d
parentec3afba5d48427801a490ba91e6ce6d0bd3bfae9
Overhaul `MacArgs::Eq`.

The value in `MacArgs::Eq` is currently represented as a `Token`.
Because of `TokenKind::Interpolated`, `Token` can be either a token or
an arbitrary AST fragment. In practice, a `MacArgs::Eq` starts out as a
literal or macro call AST fragment, and then is later lowered to a
literal token. But this is very non-obvious. `Token` is a much more
general type than what is needed.

This commit restricts things, by introducing a new type `MacArgsEqKind`
that is either an AST expression (pre-lowering) or an AST literal
(post-lowering). The downside is that the code is a bit more verbose in
a few places. The benefit is that makes it much clearer what the
possibilities are (though also shorter in some other places). Also, it
removes one use of `TokenKind::Interpolated`, taking us a step closer to
removing that variant, which will let us make `Token` impl `Copy` and
remove many "handle Interpolated" code paths in the parser.

Things to note:
- Error messages have improved. Messages like this:
  ```
  unexpected token: `"bug" + "found"`
  ```
  now say "unexpected expression", which makes more sense. Although
  arbitrary expressions can exist within tokens thanks to
  `TokenKind::Interpolated`, that's not obvious to anyone who doesn't
  know compiler internals.
- In `parse_mac_args_common`, we no longer need to collect tokens for
  the value expression.
clippy_utils/src/ast_utils.rs