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

List:       kde-core-devel
Subject:    Re: KShell::envExpand
From:       "Dawit A." <adawit () kde ! org>
Date:       2003-09-01 17:10:09
[Download RAW message or body]

On Sunday 31 August 2003 17:36, Oswald Buddenhagen wrote:
> 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.

I did not know about that class :) That might be an even better solution.

-- 
Regards,
Dawit A.
"Preach what you practice, practice what you preach"
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic