[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: KShell::envExpand
From:       Oswald Buddenhagen <ossi () kde ! org>
Date:       2003-08-31 21:36:37
[Download RAW message or body]

On Sun, Aug 31, 2003 at 06:40:07PM +0200, Dirk Mueller wrote:
> On Sunday 31 August 2003 09:41, Dawit A. wrote:
> 
> > Any objections to me adding the attached utility function to expand
> > environment variables to KShell ?
> 
> can you explain more what it is supposed to do?
> 
> - why does it only check for an embedded shell variable at the first position 
> in the string?
> 
> - why does it try to find a termination at '/' ? shell variables terminate at 
> a lot of characters, like for example simple spaces ' '. 
> 
> - the documentation is quite unclear IMHO. 
> 
> 
> Given the extremely limited amount of use of this function right now I would 
> vote against inclusion in kdelibs. 
> 
my thinking.
i'd go for something flexible built on KMacroExpander. i'll scribble
something up tomorrow.

greetings

-- 
Hi! I'm a .signature virus! Copy me into your ~/.signature, please!
--
Chaos, panic, and disorder - my work here is done.
[prev in list] [next in list] [prev in thread] [next in thread]