[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: Re: [Panel-devel] Service Runner Patch
From: fonz2591 <fonz2591 () gmail ! com>
Date: 2007-02-25 23:53:14
Message-ID: 7901c4610702251553k1846e3a9mdd66fe9fa40b191d () mail ! gmail ! com
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
Oops...
I actually stopped usig iostream, but forgot to take off the header :-/.
Im applying for a svn account, so hopefully I don't hafto deal w/ patches
:-) .
I just realized I made a slight error...
Someone should add && service->type() == "Application" to the return
statement.
On 2/25/07, Aaron J. Seigo <aseigo@kde.org> wrote:
>
> On February 25, 2007, fonz2591 wrote:
> > Here is a patch to allow KRunner to run programs based on the "Name="
> tag
> > in their .desktop files.
> >
> > I also included a .desktop file to test this feature.
> > Place it in KDE4 Directory/share/applications/kde4/ .
> > Test it by typing Test App into KRunner. It should launch konsole.
>
> +#include <iostream>
> +using namespace std;
>
> woops =) some debug stuff left ... btw, KDebug is often easier to use with
> Qt/KDE objects.
>
> + // i reccomend not trying 'service != 0' (its not worth the errors)
> + return (service && service->name() == term);
>
> i just looked and kserviceptr does actually have a converstion to bool.
>
> so "return service;" should workd, though "return !service.isNull()" might
> be
> more readable.
>
> --
> Aaron J. Seigo
> humru othro a kohnu se
> GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA EE75 D6B7 2EB1 A7F1 DB43
>
> Full time KDE developer sponsored by Trolltech (http://www.trolltech.com)
>
> _______________________________________________
> Panel-devel mailing list
> Panel-devel@kde.org
> https://mail.kde.org/mailman/listinfo/panel-devel
>
>
>
--
// Aaron
[Attachment #5 (text/html)]
Oops...<br>I actually stopped usig iostream, but forgot to take off the header \
:-/.<br>Im applying for a svn account, so hopefully I don't hafto deal w/ patches \
:-) .<br><br>I just realized I made a slight error...<br> Someone should add <span \
style="font-style: italic;">&& service->type() == \
"Application"</span> to the return statement.<br><br><div><span \
class="gmail_quote">On 2/25/07, <b class="gmail_sendername">Aaron J. Seigo </b> \
<<a href="mailto:aseigo@kde.org">aseigo@kde.org</a>> wrote:</span><blockquote \
class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt \
0pt 0.8ex; padding-left: 1ex;">On February 25, 2007, fonz2591 wrote: <br>> Here is \
a patch to allow KRunner to run programs based on the "Name=" tag<br>> \
in their .desktop files.<br>><br>> I also included a .desktop file to test this \
feature.<br>> Place it in KDE4 Directory/share/applications/kde4/ . <br>> Test \
it by typing Test App into KRunner. It should launch \
konsole.<br><br>+#include <iostream><br>+using namespace std;<br><br>woops =) \
some debug stuff left ... btw, KDebug is often easier to use with<br> Qt/KDE \
objects.<br><br>+ // i reccomend not trying 'service != \
0' (its not worth the errors)<br>+ return (service \
&& service->name() == term);<br><br>i just looked and kserviceptr does \
actually have a converstion to bool. <br><br>so "return service;" should \
workd, though "return !service.isNull()" might be<br>more \
readable.<br><br>--<br>Aaron J. Seigo<br>humru othro a kohnu se<br>GPG Fingerprint: \
8B8B 2209 0C6F 7C47 B1EA EE75 D6B7 2EB1 A7F1 DB43 <br><br>Full time KDE \
developer sponsored by Trolltech (<a \
href="http://www.trolltech.com">http://www.trolltech.com</a>)<br><br>_______________________________________________<br>Panel-devel \
mailing list<br><a href="mailto:Panel-devel@kde.org"> Panel-devel@kde.org</a><br><a \
href="https://mail.kde.org/mailman/listinfo/panel-devel">https://mail.kde.org/mailman/listinfo/panel-devel</a><br><br><br></blockquote></div><br><br \
clear="all"><br>-- <br>// Aaron
_______________________________________________
Panel-devel mailing list
Panel-devel@kde.org
https://mail.kde.org/mailman/listinfo/panel-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic