Review Request 121930: [OS X] improvements to KSharedData

René J.V. Bertin rjvbertin at gmail.com
Fri Jan 9 14:19:21 GMT 2015


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

(Updated Jan. 9, 2015, 3:19 p.m.)


Review request for KDE Software on Mac OS X, kdelibs and Allen Winter.


Repository: kdelibs


Description
-------

This patch improves KSharedData on 2 points:

- It enables `KSDC_THREAD_PROCESS_SHARED_SUPPORTED` on OS X because even if the OS cannot do timeouts on mutex locking, it does have Posix mutexes (pthreads). I don't know why this was deactivated explicitly on OS X (do you remember, Allan?), but haven't seen issues with KSDC_THREAD_PROCESS_SHARED_SUPPORTED - for now.

- OS X doesn't have `posix_fallocate()`, but an emulation of this function is available in the Mozilla code (reference found on StackOverflow). The code seems to be license-compatible, so I removed the code for non-OS X platforms, and include it in `kshareddatacache_p.h`. Again, this seems to work.


Diffs (updated)
-----

  kdecore/util/kshareddatacache_p.h 931de4d 
  kdecore/util/posix_fallocate_mac.h PRE-CREATION 

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


Testing
-------

On OS X 10.9.5 with kdelibs 4.14.4 and KDE PIM 4.13.3 (I use KMail as my default MUA).


Thanks,

René J.V. Bertin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20150109/e3bc43b9/attachment.htm>


More information about the kde-core-devel mailing list