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

List:       kde-core-devel
Subject:    Re: Proposal: virtualify KConfigBase::parseConfigFiles()
From:       Charles <charles () altair ! dhs ! org>
Date:       2000-07-10 16:21:31
[Download RAW message or body]

On Thu, 06 Jul 2000, you wrote:
> On Thu, 6 Jul 2000, Charles wrote:
> 
> > 
> > I'm using KSimpleConfig to create a stupid rediculous file format, and I want
> > it to support KIO-files.  Well, rather than breaking modularity, I just derive
> > from KSimpleConfig, and override the destructor and parseConfigFiles so that it
> > will save and open via the NetAccess class.
> > 
> > Problem: it won't work, parseConfigFiles is non-virtual, and there would be no
> > other way to read the file off the network.
> > 
> > So, I propose that we make that method virtual to allow this. (Since it's
> > already protected, it seems illogical for it to not be virtual in the first
> > place :)
> > 
> 
> Why don't you simply implement a KConfigBackEnd?
Overkill.

This adds lots of complexity, when a far simpler solution would be to
virtualify a function that already needs it.

-Charles


> 
> -- 
>   Preston Brown
>   pbrown@redhat.com                                Red Hat, Inc.
--

Charles - charles@kde.org
while(1) fork();

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