on master help sign in
make :toggle-option print the new value (#6774)
7706ff7Aleksey Kuznetsov
helix-term/src/commands/typed.rs+6−4
@@ -1770,12 +1770,12 @@ fn toggle_option(
1770 let pointer = format!("/{}", key.replace('.', "/"));
1770 let pointer = format!("/{}", key.replace('.', "/"));
1771 let value = config.pointer_mut(&pointer).ok_or_else(key_error)?;
1771 let value = config.pointer_mut(&pointer).ok_or_else(key_error)?;
1772
1772
1773− if let Value::Bool(b) = *value {
1773+ let Value::Bool(old_value) = *value else {
1774− *value = Value::Bool(!b);
1775− } else {
1776 anyhow::bail!("Key `{}` is not toggle-able", key)
1774 anyhow::bail!("Key `{}` is not toggle-able", key)
1777− }
1775+ };
1778
1776
1777+ let new_value = !old_value;
1778+ *value = Value::Bool(new_value);
1779 // This unwrap should never fail because we only replace one boolean value
1779 // This unwrap should never fail because we only replace one boolean value
1780 // with another, maintaining a valid json config
1780 // with another, maintaining a valid json config
1781 let config = serde_json::from_value(config).unwrap();
1781 let config = serde_json::from_value(config).unwrap();
@@ -1784,6 +1784,8 @@ fn toggle_option(
1784 .config_events
1784 .config_events
1785 .0
1785 .0
1786 .send(ConfigEvent::Update(config))?;
1786 .send(ConfigEvent::Update(config))?;
1787+ cx.editor
1788+ .set_status(format!("Option `{}` is now set to `{}`", key, new_value));
1787 Ok(())
1789 Ok(())
1788}
1790}
1789
1791