diff options
| author | nsfisis <nsfisis@gmail.com> | 2026-08-16 00:59:54 +0900 |
|---|---|---|
| committer | nsfisis <nsfisis@gmail.com> | 2026-08-16 01:03:45 +0900 |
| commit | 3a388b98a9aa6a14b1c7f7dc909c109cf9837800 (patch) | |
| tree | 593e342313c6e6bd3ec2943a1690b8798e0a83ae /crates/shirabe/src/dependency_resolver/solver.rs | |
| parent | aad468e8b75ffc3e87ea6dfa22c53a8299fc08da (diff) | |
| download | php-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.rs | 24 |
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 { |
