qmenumodel

Code review comment for lp://staging/~3v1n0/qmenumodel/variant-string-parser

  1. variant-string-parser
  2. Merge into trunk
Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

> Do the actions which we activate contain the parameter types for the types
> which they require? As far as I remember, it's accessible via
> g_action_group_get_action_parameter_type.
>
> Wouldn't it be better to try translate the type given to the type required
> automatically like in UnityMenuModel::changeState with
> Converter::toGVariantWithSchema

Thanks, added that too.

« Back to merge proposal