Review Request: Plasma::AbstractRunner::Syntax

Aaron Seigo aseigo at kde.org
Fri Mar 20 21:13:37 CET 2009


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/372/
-----------------------------------------------------------

(Updated 2009-03-20 13:13:37.227582)


Review request for Plasma.


Summary (updated)
-------

Allows runners to register their syntax so that it can be displayed to the user, and perhaps even used in the future to automate the process of deciding whether a runner should actually be activated or not.

:q: is used to mean "the search term" in the texts.

A class for Syntax was added rather than a more simple addSyntax("example query", "description") API in AbstractRunner so that we can extend what is associated with Syntax in future versions if we wish. It also opens the door for runners to subclass Syntax and have the syntax objects update dynamically based on settings or environment.

apidox are missing.


Diffs
-----

  trunk/KDE/kdelibs/plasma/abstractrunner.h 938702 
  trunk/KDE/kdelibs/plasma/abstractrunner.cpp 938702 
  trunk/KDE/kdelibs/plasma/runnermanager.cpp 938756 

Diff: http://reviewboard.kde.org/r/372/diff


Testing
-------


Thanks,

Aaron



More information about the Plasma-devel mailing list