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

List:       kde-commits
Subject:    Re: KDE/kdebase/kioslave/ldap
From:       Nadeem Hasan <nhasan () nadmm ! com>
Date:       2006-01-28 21:44:22
Message-ID: 200601281644.22729.nhasan () nadmm ! com
[Download RAW message or body]

On Saturday 28 January 2006 2:11 pm, Szombathelyi György wrote:
> SVN commit 503312 by gyurco:
>
> It's an overkill, this Url only has protocol://hostname

> -  ret = ldap_initialize( &mLDAP, Qt::escape(Url.prettyURL()).toUtf8() );
> +  ret = ldap_initialize( &mLDAP, Url.url().toLatin1() );

It is still better to keep it. Think about the situation when someone uses 
this ioslave as a base to create a new one.

Regards.

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