Code review comment for lp:~elementary-pantheon/switchboard/switchboard-gmodule

Revision history for this message
Rico Tzschichholz (ricotz) wrote :

I already mentioned my concerns about the Peas usage.
Look into using Peas.ExtensionBase and Peas.Activatable.

I would expect something like the following and its consequences, but I am not that familiar with peas yet.

    public abstract class Switchboard.Plug : Peas.ExtensionBase, Peas.Activatable {
    }

    exts = new Peas.ExtensionSet (engine, typeof (Switchboard.Plug), null);

« Back to merge proposal