GlimmerLabs / gimp-dbus

A Gimp plugin that provides some D-Bus services for gimp. (Reveals the PDB, among other things.)
GNU Lesser General Public License v3.0
4 stars 2 forks source link

asize parameter in gimp_dbus_gimp_param_to_g_variant #13

Open rebelsky opened 11 years ago

rebelsky commented 11 years ago

gimp_dbus_param_to_g_variant has a parameter called asize that is a pointer to an integer. It's not at all clear why we're using a pointer rather than just an integer. Explore the issue and either fix or document.

rebelsky commented 11 years ago

In addition, it's a stupid parameter to have if the GimpParam is not an array. It probably makes sense to split this into two procedures.