[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-commits
Subject: Re: KDE/kdelibs/kdecore
From: Christian Ehrlicher <Ch.Ehrlicher () gmx ! de>
Date: 2010-11-09 20:14:29
Message-ID: 4CD9ABA5.9020904 () gmx ! de
[Download RAW message or body]
Am 09.11.2010 21:06, schrieb David Faure:
> On Tuesday 09 November 2010, Christian Ehrlicher wrote:
>> SVN commit 1194753 by chehrlic:
>>
>> compile++
>
> Ouch, I could have at least done a pass over the _win.cpp files visually.
> Sorry for the additional work, and thanks for the fix.
>
> I'm also guilty for the qbytearray usage which made you add this in
> kstandarddirs.cpp:
> gethostname(hostname, 255);
> const QString localkdedir = m_prefixes.first();
> + // this will not work on windows when localkdedir contains
> + // characters not encodeable in the current locale
> QByteArray dir = QFile::encodeName(localkdedir);
> dir += type;
> dir += '-';
>
> I forgot that the best representation of a path on Windows is a QString
> (while on unix one could argue that it's qbytearray, but none of the qt api
> takes that anyway).
>
> Shall I change it back to QString?
> [with the risk to break compilation again, although I'll be careful :)]
>
It would be nice, yes :)
Thx,
Christian
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic