Skip to content

Conversation

smanilov
Copy link
Contributor

The order might have been reversed at some point, leading to the two chapters talking about each other in the wrong order.

The order might have been reversed at some point, leading to the two
chapters talking about each other in the wrong order.
@smanilov
Copy link
Contributor Author

Just to make it clearer for review: at the moment, instantiating_binders comes first (chapter 48.1). It refers to param_ty_const_regions (chapter 50.2) indirectly via "As discussed in a previous chapter, RegionKind has two variants for representing generic parameters, ReLateParam and ReEarlyParam.".

I searched in the docs and saw no other references to ReLateParam and ReEarlyParam, so I proposed this change.

@BoxyUwU
Copy link
Member

BoxyUwU commented May 20, 2025

Possibly we should move the "the ty module" chapter before the binders chapter, but I'm not sure and regardless it seems like an improvement to make the wording be chapter order insensitive

@BoxyUwU BoxyUwU merged commit c693842 into rust-lang:master May 20, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants