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

List:       kde-devel
Subject:    PATCH: Re: IKWS config file doubling backslashes
From:       Malte.Starostik () t-online ! de (Malte Starostik)
Date:       2000-09-26 15:11:21
[Download RAW message or body]

On Die, 26 Sep 2000, Malte Starostik wrote:
> On Die, 26 Sep 2000, Malte Starostik wrote:
> > Hi,
> >
> > everytime ~/.kde/share/config/kuriikwsfilterrc is saved, the backslashes
> > denoting parameters are doubled. That causes the superfluous backslashes
> > to be interpreted as part of the URI. Could that be a cause of the
> > KConfig changes?
>
> Ehhhm.......
> and this is from klipper's config dialog:
> ^http:\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\/\\\\
>\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\/ At least I
> know why it doesn't react on URLs anymore

Can someone review this patch please, the translation from \ -> \\ was 
performed but not vice versa. I've got absolutely no clue what this could 
break but AFAICT it fixes the problem described above. As Sven Radej wrote on 
core-devel, I also got a 8.5 MB konquerorrc full of \\\\\\\\ after only 4-5 
hours of usage (since I last deleted it). With that config, konqueror grew to 
125MB in memory regardless of what it was displaying.
-Malte


["kconfig.diff" (text/plain)]

Index: kconfigbackend.cpp
===================================================================
RCS file: /home/kde/kdelibs/kdecore/kconfigbackend.cpp,v
retrieving revision 1.39
diff -u -b -r1.39 kconfigbackend.cpp
--- kconfigbackend.cpp	2000/09/25 03:53:04	1.39
+++ kconfigbackend.cpp	2000/09/26 15:04:16
@@ -81,6 +81,9 @@
            case 'r':
               *r++ = '\r';
               break;
+           case '\\':
+              *r++ = '\\';
+              break;
            default:
               *r++ = '\\';
               *r++ = *str;

>> Visit http://master.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<


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