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/command/config_command.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/command/config_command.rs')
| -rw-r--r-- | crates/shirabe/src/command/config_command.rs | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/crates/shirabe/src/command/config_command.rs b/crates/shirabe/src/command/config_command.rs index 3d92de4b..504a0f11 100644 --- a/crates/shirabe/src/command/config_command.rs +++ b/crates/shirabe/src/command/config_command.rs @@ -38,16 +38,15 @@ pub struct ConfigCommand { config_file: std::cell::RefCell<Option<std::rc::Rc<std::cell::RefCell<JsonFile>>>>, config_source: std::cell::RefCell<Option<JsonConfigSource>>, - pub(crate) auth_config_file: - std::cell::RefCell<Option<std::rc::Rc<std::cell::RefCell<JsonFile>>>>, - pub(crate) auth_config_source: std::cell::RefCell<Option<JsonConfigSource>>, + auth_config_file: std::cell::RefCell<Option<std::rc::Rc<std::cell::RefCell<JsonFile>>>>, + auth_config_source: std::cell::RefCell<Option<JsonConfigSource>>, } impl_php_class!(ConfigCommand, r"Composer\Command\ConfigCommand"); impl ConfigCommand { /// List of additional configurable package-properties - pub(crate) const CONFIGURABLE_PACKAGE_PROPERTIES: &'static [&'static str] = &[ + const CONFIGURABLE_PACKAGE_PROPERTIES: &'static [&'static str] = &[ "name", "type", "description", @@ -77,7 +76,7 @@ impl ConfigCommand { command } - pub(crate) fn handle_single_value( + fn handle_single_value( &self, key: &str, callbacks: &(ValidatorFn, NormalizerFn), @@ -143,7 +142,7 @@ impl ConfigCommand { Ok(()) } - pub(crate) fn handle_multi_value( + fn handle_multi_value( &self, key: &str, callbacks: &(ValidatorFn, NormalizerFn), @@ -181,7 +180,7 @@ impl ConfigCommand { } /// Display the contents of the file in a pretty formatted way - pub(crate) fn list_configuration( + fn list_configuration( &self, contents: PhpMixed, raw_contents: PhpMixed, |
