Character-Runner
Aaron J. Seigo
aseigo at kde.org
Fri Apr 9 00:57:10 CEST 2010
On April 8, 2010, Anton Kreuzkamp wrote:
> I would appreciate it if you could review it and possibly integrate it into
> Trunk. Thanks.
looks good. two comments on it:
* context.query() is already trim()'d, so you don't have to re-trim() them.
* in reloadConfiguration, the runner should create one or more QuerySyntax
objects and register them with addSyntax or setSyntaxes.
other than that, if you'd like to maintain this in KDE's svn, that'd be cool.
you'll need an svn account[1] and commit to maintaining the plugin. it would
go into kdeplasma-addons/runners.
[1] http://techbase.kde.org/Contribute/Get_a_SVN_Account
--
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA EE75 D6B7 2EB1 A7F1 DB43
KDE core developer sponsored by Qt Development Frameworks
More information about the Plasma-devel
mailing list