CascadiaSettings::LoadAll(true) config file content compares to itself #891

Open
opened 2026-01-30 22:08:30 +00:00 by claunia · 0 comments
Owner

Originally created by @bef178 on GitHub (May 9, 2019).

in CascadiaSettingsSerialization.cpp (how to cite existing code)

fileData => actualData => root => obj => resultPtr => json => serializedSettings
then comparing actualData and serializedSettings

Looks like during the transformation only default key bindings are added, which is not configurable and won't be saved. So I believe the config content compares to itself (format change?).

Does it expect comparing the config file content to in-memory in-using config?
Or, does it expect add default config and explicitly write default configs to config file?

Please correct me if I miss something.

1

Originally created by @bef178 on GitHub (May 9, 2019). in CascadiaSettingsSerialization.cpp (how to cite existing code) fileData => **actualData** => root => obj => resultPtr => json => **serializedSettings** then comparing actualData and serializedSettings Looks like during the transformation only default key bindings are added, which is not configurable and won't be saved. So I believe the config content compares to itself (format change?). Does it expect comparing the config file content to in-memory in-using config? Or, does it expect add default config and explicitly write default configs to config file? Please correct me if I miss something. ![1](https://user-images.githubusercontent.com/2749912/57459472-e5146480-72a5-11e9-94c0-dab9c54ca7ca.png)
claunia added the Issue-Question label 2026-01-30 22:08:30 +00:00
Sign in to join this conversation.
No Label Issue-Question
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#891