Improve error message handling for theme loading failures (#5073)

The error messages for a theme that failed to be deserialized (or
otherwise failed to load) were covered up by the context/with_context
calls:

* The log message for a bad theme configured in config.toml would only
  say "Failed to deserilaize theme"
* Selecting a bad theme via :theme would show "Theme does not exist"

With these changes, we let the TOML deserializer errors bubble up, so
the error messages can now say the line number of a duplicated
key - and that key's name - when a theme fails to load because of a
duplicated key.

Providing a theme which does not exist to :theme still gives a helpful
error message: "No such file or directory."
pull/5164/head
Michael Davis 2 years ago committed by GitHub
parent 012fc12f97
commit db939801eb
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

@ -777,7 +777,7 @@ fn theme(
.editor .editor
.theme_loader .theme_loader
.load(theme_name) .load(theme_name)
.with_context(|| "Theme does not exist")?; .map_err(|err| anyhow::anyhow!("Could not load theme: {}", err))?;
if !(true_color || theme.is_16_color()) { if !(true_color || theme.is_16_color()) {
bail!("Unsupported theme: theme requires true color support"); bail!("Unsupported theme: theme requires true color support");
} }

@ -136,8 +136,9 @@ impl Loader {
// Loads the theme data as `toml::Value` first from the user_dir then in default_dir // Loads the theme data as `toml::Value` first from the user_dir then in default_dir
fn load_toml(&self, path: PathBuf) -> Result<Value> { fn load_toml(&self, path: PathBuf) -> Result<Value> {
let data = std::fs::read(&path)?; let data = std::fs::read(&path)?;
let value = toml::from_slice(data.as_slice())?;
toml::from_slice(data.as_slice()).context("Failed to deserialize theme") Ok(value)
} }
// Returns the path to the theme with the name // Returns the path to the theme with the name

Loading…
Cancel
Save