[Kde-bindings] KDE/kdebindings

[Richard Dale]

On Friday 05 February 2010 07:27:06 pm Arno Rehn wrote:
> SVN commit 1085699 by arnorehn:
> 
> Add the TerminalInterface classes.
> 
> CCMAIL: kde-bindings at 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 at kde.org
> https://mail.kde.org/mailman/listinfo/kde-bindings
>