The addition of compizconfigTypeToVariantType makes no sense.
We already have a tight coupling between: TYPE x; and: g_variant_iter_loop (&iter, TYPESTRING, &x)
Using: g_variant_iter_loop (&iter, someVariable, &x) only makes the code harder to read, with looser coupling and weaker cohesion.
Please remove compizconfigTypeToVariantType and just use: g_variant_iter_loop (&iter, "X", &x)
« Back to merge proposal
The addition of compizconfigTyp eToVariantType makes no sense.
We already have a tight coupling between: iter_loop (&iter, TYPESTRING, &x)
TYPE x;
and:
g_variant_
Using: iter_loop (&iter, someVariable, &x)
g_variant_
only makes the code harder to read, with looser coupling and weaker cohesion.
Please remove compizconfigTyp eToVariantType and just use: iter_loop (&iter, "X", &x)
g_variant_