Review: Needs Information
On Mon, 2011-02-14 at 03:46 +0000, Luke Yelavich wrote:
> +     /* Not all indicator entries have a get_accessible_desc method, such as
> +        indicator-application entries */
> +     if (class->get_accessible_desc != NULL) {
> +             entry->accessible_desc = class->get_accessible_desc(io);
> +     }

I'm not sure why we need to update the entry here, it seems that should
be done internally in the IndicatorObject no?

  review needsinformation

-- 
https://code.launchpad.net/~themuso/indicator-applet/accessible-desc/+merge/49586
Your team ayatana-commits is subscribed to branch lp:indicator-applet.

_______________________________________________
Mailing list: https://launchpad.net/~ayatana-commits
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~ayatana-commits
More help   : https://help.launchpad.net/ListHelp

Reply via email to