aboutsummaryrefslogtreecommitdiffhomepage
path: root/crates/shirabe/src/dependency_resolver/solver.rs
diff options
context:
space:
mode:
authornsfisis <nsfisis@gmail.com>2026-08-16 00:59:54 +0900
committernsfisis <nsfisis@gmail.com>2026-08-16 01:03:45 +0900
commit3a388b98a9aa6a14b1c7f7dc909c109cf9837800 (patch)
tree593e342313c6e6bd3ec2943a1690b8798e0a83ae /crates/shirabe/src/dependency_resolver/solver.rs
parentaad468e8b75ffc3e87ea6dfa22c53a8299fc08da (diff)
downloadphp-shirabe-3a388b98a9aa6a14b1c7f7dc909c109cf9837800.tar.gz
php-shirabe-3a388b98a9aa6a14b1c7f7dc909c109cf9837800.tar.zst
php-shirabe-3a388b98a9aa6a14b1c7f7dc909c109cf9837800.zip
refactor: narrow pub(crate) items to private
Porting mapped every PHP `protected` member onto `pub(crate)`, which is wider than nearly all of them need. Each item demoted here is reached only from the module that defines it, so the crate-wide visibility conveyed nothing. Every `pub(crate)` that survives has at least one reader in another module of the same crate. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Diffstat (limited to 'crates/shirabe/src/dependency_resolver/solver.rs')
-rw-r--r--crates/shirabe/src/dependency_resolver/solver.rs24
1 files changed, 12 insertions, 12 deletions
diff --git a/crates/shirabe/src/dependency_resolver/solver.rs b/crates/shirabe/src/dependency_resolver/solver.rs
index a9453a3e..cb398a0c 100644
--- a/crates/shirabe/src/dependency_resolver/solver.rs
+++ b/crates/shirabe/src/dependency_resolver/solver.rs
@@ -26,25 +26,25 @@ use shirabe_semver::constraint::AnyConstraint;
#[derive(Debug)]
pub struct Solver {
- pub(crate) policy: std::rc::Rc<dyn PolicyInterface>,
- pub(crate) pool: std::rc::Rc<std::cell::RefCell<Pool>>,
+ policy: std::rc::Rc<dyn PolicyInterface>,
+ pool: std::rc::Rc<std::cell::RefCell<Pool>>,
- pub(crate) rules: RuleSet,
+ rules: RuleSet,
- pub(crate) watch_graph: RuleWatchGraph,
- pub(crate) decisions: Decisions,
- pub(crate) fixed_map: IndexMap<i64, BasePackageHandle>,
+ watch_graph: RuleWatchGraph,
+ decisions: Decisions,
+ fixed_map: IndexMap<i64, BasePackageHandle>,
- pub(crate) propagate_index: i64,
+ propagate_index: i64,
/// Pairs of `(literals, level)` — PHP indexes into these with the BRANCH_* constants.
- pub(crate) branches: Vec<(Vec<i64>, i64)>,
- pub(crate) problems: Vec<Problem>,
- pub(crate) learned_pool: Vec<Vec<std::rc::Rc<std::cell::RefCell<Rule>>>>,
- pub(crate) learned_why: IndexMap<String, i64>,
+ branches: Vec<(Vec<i64>, i64)>,
+ problems: Vec<Problem>,
+ learned_pool: Vec<Vec<std::rc::Rc<std::cell::RefCell<Rule>>>>,
+ learned_why: IndexMap<String, i64>,
pub test_flag_learned_positive_literal: bool,
- pub(crate) io: std::rc::Rc<std::cell::RefCell<dyn IOInterface>>,
+ io: std::rc::Rc<std::cell::RefCell<dyn IOInterface>>,
}
impl Solver {