[Kde-pim] Review Request 126884: Check for and replace strcasestr if neccessary

Andre Heinecke aheinecke at intevation.de
Mon Jan 25 17:18:34 GMT 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126884/
-----------------------------------------------------------

Review request for KDEPIM-Libraries and Volker Krause.


Repository: kmime


Description
-------

Neccessary at least for Windows (with mingw-3).
This renames the TimezoneChecks.cmake file to a more generic name so that it can be better extended if more checks are needed.

The replacement function was taken from libreplace as part of QtWebEngine. (I just grepped for strcasecmp in qt)
There is also a replacement in Kdewin but this copies the strings which I found wasteful.
The probably highly optimized version from Kde4 kdelibs/kdecore/fakes.c I've found to complicated ;-)


Diffs
-----

  cmake/CheckTimezone.cmake adc1937 
  cmake/ConfigureChecks.cmake PRE-CREATION 
  src/CMakeLists.txt ef86420 
  src/config-kmime.h.cmake 7b7ef6d 
  src/kmime_util.cpp ed002b9 

Diff: https://git.reviewboard.kde.org/r/126884/diff/


Testing
-------

Compiled under Debian GNU/Linux and for Windows with mingw.


Thanks,

Andre Heinecke

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/


More information about the kde-pim mailing list