[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-bindings
Subject: Re: [Kde-bindings] KDE/kdebindings
From: Richard Dale <rdale () foton ! es>
Date: 2010-02-06 15:46:31
Message-ID: 201002061546.31477.rdale () foton ! es
[Download RAW message or body]
On Friday 05 February 2010 07:27:06 pm Arno Rehn wrote:
> SVN commit 1085699 by arnorehn:
>
> Add the TerminalInterface classes.
>
> CCMAIL: kde-bindings@kde.org
>
>
> M +2 -0 ruby/korundum/src/lib/KDE/korundum4.rb
> M +3 -0 smoke/kparts/kparts_includes.h
> M +2 -0 smoke/kparts/smokeconfig.xml
>
>
> --- trunk/KDE/kdebindings/ruby/korundum/src/lib/KDE/korundum4.rb
> #1085698:1085699 @@ -573,6 +573,8 @@
> now = classname.sub(/^K?(?=[A-Z])/,'KDE::')
> elsif classname =~ /^K/ && classname !~ /::/
> now = classname.sub(/(^K)/, 'KDE::')
> + elsif classname =~ /^TerminalInterface/
> + now = "KDE::#{classname}"
> end
> now
> end)
Shouldn't this really be in a namespace/module of 'Konsole' - that is what the
konsole app itself uses? In fact I'm not sure why the TerminalInterface
classes aren't in a namespace in the first place.
-- Richard
> --- trunk/KDE/kdebindings/smoke/kparts/kparts_includes.h #1085698:1085699
> @@ -13,3 +13,6 @@
> #include <kparts/browserrun.h>
> #include <kparts/statusbarextension.h>
> #include <kparts/browseropenorsavequestion.h>
> +
> +#include <kde_terminal_interface.h>
> +#include <kde_terminal_interface_v2.h>
> --- trunk/KDE/kdebindings/smoke/kparts/smokeconfig.xml #1085698:1085699
> @@ -64,5 +64,7 @@
> <class>KParts::ReadOnlyPart</class>
> <class>KParts::ReadWritePart</class>
> <class>KParts::StatusBarExtension</class>
> + <class>TerminalInterface</class>
> + <class>TerminalInterfaceV2</class>
> </classList>
> </config>
> _______________________________________________
> Kde-bindings mailing list
> Kde-bindings@kde.org
> https://mail.kde.org/mailman/listinfo/kde-bindings
>
_______________________________________________
Kde-bindings mailing list
Kde-bindings@kde.org
https://mail.kde.org/mailman/listinfo/kde-bindings
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic