Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Abstract away GSettings #186

Merged

Conversation

jpnurmi
Copy link
Contributor

@jpnurmi jpnurmi commented Dec 12, 2021

This is a preparation step for #183 i.e. migrating from GSettings 0.1
to 0.2.

GSettings 0.1 and 0.2 are vastly different because the FFI-based 0.1 is
synchronous, whereas the D-Bus-based 0.2 is asynchronous. Furthermore,
0.1 has separate getters for different primitive types, whereas 0.2 has
only one getter that returns a DBusValue.

In order to make the migration from GSettings 0.1 to 0.2 easier, hide
GSettings instances from the models and use nullable setting values to
eliminate the assumption that setting values are immediately available.

This is a preparation step for ubuntu-flutter-community#183 i.e. migrating from GSettings 0.1
to 0.2.

GSettings 0.1 and 0.2 are vastly different because the FFI-based 0.1 is
synchronous, whereas the D-Bus-based 0.2 is asynchronous. Furthermore,
0.1 has separate getters for different primitive types, whereas 0.2 has
only one getter that returns a DBusValue.

In order to make the migration from GSettings 0.1 to 0.2 easier, hide
GSettings instances from the models and use nullable setting values to
eliminate the assumption that setting values are immediately available.
@Feichtmeier
Copy link
Member

Tyvm!

@Feichtmeier Feichtmeier merged commit e9ca4a9 into ubuntu-flutter-community:master Dec 12, 2021
@jpnurmi jpnurmi deleted the prepare-settings branch December 12, 2021 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants