[patch] KConfig on windows/msvc, simplified
Jaroslaw Staniek
js at iidea.pl
Tue Nov 6 23:45:50 GMT 2007
Oswald Buddenhagen said the following, On 2007-11-07 00:21:
> On Tue, Nov 06, 2007 at 10:56:19PM +0100, Jaroslaw Staniek wrote:
>> Uff. It's cleaned version of the patch, without part related to cleaning up
>> of identifiers.
>>
> thanks, that's much simpler to review.
> note that you still have some weird whitespace changes in there.
ok, but where's that?
> however, you entirely failed to respond to that:
>
>>> you also did another significant change: the read/write functions
>>> for the directly supported data types are no more template
>>> spcializations.
>>>
>>> i'm wondering how the changes behave when isolated from each other.
Depending which part of change you keep, with msvc you'll get errors related
to QVariant internals or internal compiler's error.
The change is source compatible, except for qbytearray overloads (but usage of
qbytearray keys came from the kconfig merge in
http://websvn.kde.org/?view=rev&revision=721704).
--
regards / pozdrawiam, Jaroslaw Staniek
Sponsored by OpenOffice Polska (http://www.openoffice.com.pl/en) to work on
Kexi & KOffice: http://www.kexi.pl/en, http://www.koffice.org
KDE3 & KDE4 Libraries for MS Windows: http://kdelibs.com, http://www.kde.org
More information about the kde-core-devel
mailing list