Rollup merge of #124687 - fee1-dead-contrib:private-clauses, r=compiler-errors

compiler-errors · web-flow · commit d67586e293e5 · 2024-05-03T22:30:17.000-04:00
Make `Bounds.clauses` private

Construct it through `Bounds::default()`, then consume the clauses via the method `Bounds::clauses()`.

This helps with effects desugaring where `clauses()` is not only the clauses within the `clauses` field.
diff --git a/compiler/rustc_hir_analysis/src/bounds.rs b/compiler/rustc_hir_analysis/src/bounds.rs
@@ -23,7 +23,7 @@ use rustc_span::Span;
 /// include the self type (e.g., `trait_bounds`) but in others we do not
 #[derive(Default, PartialEq, Eq, Clone, Debug)]
 pub struct Bounds<'tcx> {
-    pub clauses: Vec<(ty::Clause<'tcx>, Span)>,
+    clauses: Vec<(ty::Clause<'tcx>, Span)>,
 }
 
 impl<'tcx> Bounds<'tcx> {

Original file line number	Diff line number	Diff line change
`@@ -23,7 +23,7 @@ use rustc_span::Span;`
`23`	`23`	/// include the self type (e.g., `trait_bounds`) but in others we do not
`24`	`24`	`#[derive(Default, PartialEq, Eq, Clone, Debug)]`
`25`	`25`	`pub struct Bounds<'tcx> {`
`26`		`- pub clauses: Vec<(ty::Clause<'tcx>, Span)>,`
	`26`	`+ clauses: Vec<(ty::Clause<'tcx>, Span)>,`
`27`	`27`	`}`
`28`	`28`
`29`	`29`	`impl<'tcx> Bounds<'tcx> {`