profile: switch DefaultResolution/ActiveResolution over to an object path

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
This commit is contained in:
Peter Hutterer
2016-02-15 15:04:14 +10:00
parent f2a55bca55
commit e199278d6f
3 changed files with 34 additions and 24 deletions

View File

@@ -117,24 +117,22 @@ static int ratbagd_profile_get_active_resolution(sd_bus *bus,
sd_bus_error *error)
{
struct ratbagd_profile *profile = userdata;
unsigned int i, n_resolutions, k = 0;
struct ratbagd_resolution *resolution;
unsigned int i;
n_resolutions = ratbag_profile_get_num_resolutions(profile->lib_profile);
for (i = 0; i < n_resolutions; ++i) {
struct ratbag_resolution *resolution;
resolution = ratbag_profile_get_resolution(profile->lib_profile, i);
for (i = 0; i < profile->n_resolutions; ++i) {
resolution = profile->resolutions[i];
if (!resolution)
continue;
if (!ratbag_resolution_is_active(resolution)) {
++k;
if (!ratbagd_resolution_is_active(resolution))
continue;
}
return sd_bus_message_append(reply, "u", k);
return sd_bus_message_append(reply, "o",
ratbagd_resolution_get_path(resolution));
}
return sd_bus_message_append(reply, "u", (unsigned int)-1);
/* Eww, we want 'maybe' types here! */
return sd_bus_message_append(reply, "o", "/");
}
static int ratbagd_profile_get_default_resolution(sd_bus *bus,
@@ -146,24 +144,22 @@ static int ratbagd_profile_get_default_resolution(sd_bus *bus,
sd_bus_error *error)
{
struct ratbagd_profile *profile = userdata;
unsigned int i, n_resolutions, k = 0;
struct ratbagd_resolution *resolution;
unsigned int i;
n_resolutions = ratbag_profile_get_num_resolutions(profile->lib_profile);
for (i = 0; i < n_resolutions; ++i) {
struct ratbag_resolution *resolution;
resolution = ratbag_profile_get_resolution(profile->lib_profile, i);
for (i = 0; i < profile->n_resolutions; ++i) {
resolution = profile->resolutions[i];
if (!resolution)
continue;
if (!ratbag_resolution_is_default(resolution)) {
++k;
if (!ratbagd_resolution_is_default(resolution))
continue;
}
return sd_bus_message_append(reply, "u", k);
return sd_bus_message_append(reply, "o",
ratbagd_resolution_get_path(resolution));
}
return sd_bus_message_append(reply, "u", (unsigned int)-1);
/* Eww, we want 'maybe' types here! */
return sd_bus_message_append(reply, "o", "/");
}
static int ratbagd_profile_set_active(sd_bus_message *m,
@@ -206,8 +202,8 @@ const sd_bus_vtable ratbagd_profile_vtable[] = {
SD_BUS_VTABLE_START(0),
SD_BUS_PROPERTY("Index", "u", NULL, offsetof(struct ratbagd_profile, index), SD_BUS_VTABLE_PROPERTY_CONST),
SD_BUS_PROPERTY("Resolutions", "ao", ratbagd_profile_get_resolutions, 0, 0),
SD_BUS_PROPERTY("ActiveResolution", "u", ratbagd_profile_get_active_resolution, 0, 0),
SD_BUS_PROPERTY("DefaultResolution", "u", ratbagd_profile_get_default_resolution, 0, 0),
SD_BUS_PROPERTY("ActiveResolution", "o", ratbagd_profile_get_active_resolution, 0, 0),
SD_BUS_PROPERTY("DefaultResolution", "o", ratbagd_profile_get_default_resolution, 0, 0),
SD_BUS_METHOD("SetActive", "", "u", ratbagd_profile_set_active, SD_BUS_VTABLE_UNPRIVILEGED),
SD_BUS_METHOD("GetResolutionByIndex", "u", "o", ratbagd_profile_get_resolution_by_index, SD_BUS_VTABLE_UNPRIVILEGED),
SD_BUS_VTABLE_END,

View File

@@ -112,3 +112,15 @@ struct ratbagd_resolution *ratbagd_resolution_free(struct ratbagd_resolution *re
return mfree(resolution);
}
bool ratbagd_resolution_is_active(struct ratbagd_resolution *resolution)
{
assert(resolution);
return ratbag_resolution_is_active(resolution->lib_resolution) != 0;
}
bool ratbagd_resolution_is_default(struct ratbagd_resolution *resolution)
{
assert(resolution);
return ratbag_resolution_is_default(resolution->lib_resolution) != 0;
}

View File

@@ -74,6 +74,8 @@ int ratbagd_resolution_new(struct ratbagd_resolution **out,
unsigned int index);
struct ratbagd_resolution *ratbagd_resolution_free(struct ratbagd_resolution *resolution);
const char *ratbagd_resolution_get_path(struct ratbagd_resolution *resolution);
bool ratbagd_resolution_is_active(struct ratbagd_resolution *resolution);
bool ratbagd_resolution_is_default(struct ratbagd_resolution *resolution);
DEFINE_TRIVIAL_CLEANUP_FUNC(struct ratbagd_resolution *, ratbagd_resolution_free);