Code review comment for lp:~charlesk/indicator-appmenu/fix-953479

Revision history for this message
Charles Kerr (charlesk) wrote :

Ryan, I made most of the modifications you suggested, but am assuming that your last line wasn't literal -- you wouldn't pass a g_variant_builder_init()'ed GVariantBuilder into g_variant_new()'s vararg list, nor return the result immediately?

« Back to merge proposal