[patch] strigi (trunk) fixes for msvc

Jarosław Staniek js at iidea.pl
Wed Jan 30 10:37:07 GMT 2008


For review:

- the main issue was that int32_t is 'signed long' and not 'int' (compare 
include/msvc/stdint.h with clucene typesdefs), so we needed to add some ifdefs

--- in for one case (cluceneindexreader.cpp) added a (ifdef'd) long long 
overload for Strigi::Variant's operator= because if we include variant.h 
header within a scope of different int32_t definition, we'll get linking error

- added casts for ambiguous overloads of AnalysisResult::addValue(), usually 
using (int32_t)

- there were also problems with windows.h/winsock2.h includes - to fix 
conflicting macros it we needed to include winsock2.h early enough
(of course we cannot alter the system headers and _wanted_ to avoid altering 
clucene headers)

- added defines like HAVE_SOCKET_H for msvc as these are always available 
thanks to kdewin32 layer


I have assumed that msvc target is the one dependent on kdewin32.

A number of above fixes will be reused for mingw target as well.

-- 
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/kexi)
  KDE Libraries for MS Windows (http://windows.kde.org)
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: strigi-msvc.patch
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20080130/df31d479/attachment.ksh>


More information about the kde-core-devel mailing list