rust/compiler/rustc_parse/src/parser/attr_wrapper.rs, branch 1.74.0

Adjust `to_attr_token_stream`.

2023-09-06T07:12:07+00:00

It uses `once` chained with `(0..self.num_calls).map(...)` followed by `.take(self.num_calls`. I found this hard to read. It's simpler to just use `repeat_with`.

Move `TokenCursor::break_last_token` into `Parser`.

2023-08-02T00:11:41+00:00

Similar to the last commit, it's more of a `Parser`-level concern than a `TokenCursor`-level concern. And the struct size reductions are nice. After this change, `TokenCursor` is as minimal as possible (two fields and two methods) which is nice.

Move `TokenCursor::num_next_calls` into `Parser` and rename it.

2023-08-02T00:07:18+00:00

It's more of a `Parser`-level concern than a `TokenCursor`-level concern. Also, `num_bump_calls` is a more accurate name, because it's incremented in `Parser::bump`.

inline format!() args up to and including rustc_middle

2023-07-30T11:18:33+00:00

Remove `desugar_doc_comments` arguments from `TokenCursor::{inlined_,}next`.

2023-07-26T02:46:43+00:00

Because it's now always `self.desugar_doc_comments`.

Simplify some conditions

2023-06-27T07:40:47+00:00

Use `Option::is_some_and` and `Result::is_ok_and` in the compiler

2023-05-24T14:20:41+00:00

Match unmatched backticks in comments in compiler/

2023-03-03T07:39:00+00:00

Remove `TokenCursorFrame`.

2023-02-02T23:06:52+00:00

The motivation here is to eliminate the `Option<(Delimiter, DelimSpan)>`, which is `None` for the outermost token stream and `Some` for all other token streams. We are already treating the innermost frame specially -- this is the `frame` vs `stack` distinction in `TokenCursor`. We can push that further so that `frame` only contains the cursor, and `stack` elements contain the delimiters for their children. When we are in the outermost token stream `stack` is empty, so there are no stored delimiters, which is what we want because the outermost token stream *has* no delimiters. This change also shrinks `TokenCursor`, which shrinks `Parser` and `LazyAttrTokenStreamImpl`, which is nice.

compiler: remove unnecessary imports and qualified paths

2022-12-10T17:45:34+00:00