Re: [SyncEvolution] [REVIEW REQUEST 2/7] GDBus GIO: add variant append support
On Mo, 2012-01-30 at 09:23 +0100, Mikel Astiz wrote:
> I checked with dbus-monitor, the resulting D-Bus message has
"variant
> string" as type of value, as intended.
>
Well, I just guessed that g_variant_builder_open would accept the "v" as
well, and then tested. If you prefer we could use the type of the
variant value, which would involve opening the builder inside the
append_visitor. As you prefer.
I'm fine with the code as it is. I was merely trying to understand
it ;-}
--
Best Regards, Patrick Ohly
The content of this message is my personal opinion only and although
I am an employee of Intel, the statements I make here in no way
represent Intel's position on the issue, nor am I authorized to speak
on behalf of Intel on this matter.