[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-bindings
Subject: Re: [Kde-bindings] KDE/kdebindings
From: Arno Rehn <arno () arnorehn ! de>
Date: 2010-02-06 16:06:55
Message-ID: 201002061706.55467.arno () arnorehn ! de
[Download RAW message or body]
On Saturday 06 February 2010 16:46:31 Richard Dale wrote:
> 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.
That's why I put them in the KDE namespace for a start. As they are not in any
namespace in C++ either, I don't know if people actually think about whether
there could be a 'Konsole' namespace which contains the classes.
--
Arno Rehn
arno@arnorehn.de
_______________________________________________
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