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

Andre Heinecke aheinecke at intevation.de
Fri Feb 12 15:13:02 GMT 2016


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

(Updated Feb. 12, 2016, 3:13 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDEPIM-Libraries and Volker Krause.


Changes
-------

Submitted with commit cd6220e8c32b75791e27a8323a1552c82767811a by Andre Heinecke to branch master.


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