Skip to content

Conversation

lnicola
Copy link
Member

@lnicola lnicola commented May 9, 2023

No description provided.

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 9, 2023
let p = SourceFile::parse(&data);
assert!(p.errors.is_empty());
assert_eq!(p.tree().syntax.text_range().len(), 352474.into());
assert_eq!(p.tree().syntax.text_range().len(), 352458.into());
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because of the fmt changes, maybe?

matthiaskrgr and others added 5 commits May 9, 2023 07:41
This reverts commit abc0660118cc95f47445fd33502a11dd448f5968.
This function/lang_item was introduced in #104321 as a temporary workaround of future lowering.
The usage and need for it went away in #104833.
After a bootstrap update, the function itself can be removed from `std`.
@lnicola
Copy link
Member Author

lnicola commented May 9, 2023

Dropped the two commits from rust-lang/rust#109002 because they break building against the stable sysroot.

@lnicola
Copy link
Member Author

lnicola commented May 9, 2023

Actually, according to #13676 (comment), it seems we can't drop commits that way.

@lnicola lnicola closed this May 9, 2023
@lnicola lnicola deleted the sync-from-rust branch May 9, 2023 05:08
@Veykril
Copy link
Member

Veykril commented May 9, 2023

Hmm, can we add those commits and merge them in without CI approval? Then fix them up afterwards?

@Veykril
Copy link
Member

Veykril commented May 9, 2023

Ahh nevermind I see the issue now

@lnicola
Copy link
Member Author

lnicola commented May 9, 2023

Yes, but we'll have to propagate our "fix" downstream, if it involves changing the code there 😄 .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants