mirrors/jj
1
0
Fork 1
mirror of https://github.com/martinvonz/jj.git synced 2025-01-02 09:52:10 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 7

Change with_toml_strings to incorporate_toml_string

Browse source 
It makes more sense for this function to change `self`.
This commit is contained in:
Ilya Grigoriev 2023-01-01 16:54:10 -08:00
parent 95637e252c
commit e8b21c5ce0
2 changed files with 8 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
lib/src/settings.rs
View file

@ -45,21 +45,18 @@ impl UserSettings {
UserSettings { config, timestamp }
}
pub fn with_toml_strings(
&self,
pub fn incorporate_toml_strings(
&mut self,
toml_strs: &[String],
) -> Result<UserSettings, config::ConfigError> {
) -> Result<(), config::ConfigError> {
let mut config_builder = config::Config::builder().add_source(self.config.clone());
for s in toml_strs {
config_builder =
config_builder.add_source(config::File::from_str(s, config::FileFormat::Toml));
}
let new_config = config_builder.build()?;
let timestamp = get_timestamp_config(&new_config, "user.timestamp");
Ok(UserSettings {
config: new_config,
timestamp,
})
self.config = config_builder.build()?;
self.timestamp = get_timestamp_config(&self.config, "user.timestamp");
Ok(())
}
pub fn with_repo(&self, repo_path: &Path) -> Result<RepoSettings, config::ConfigError> {

3
src/cli_util.rs
View file

@ -1590,7 +1590,8 @@ fn handle_early_args(
ui.set_pagination(crate::ui::PaginationChoice::No);
}
if !args.config_toml.is_empty() {
let settings = ui.settings().with_toml_strings(&args.config_toml)?;
let mut settings = ui.settings().clone();
settings.incorporate_toml_strings(&args.config_toml)?;
ui.reset(settings);
}
Ok(())