On 02/05/2012 07:21 PM, Francois Tigeot wrote:
sal/osl/unx/semaphor.c contains two separate implementations of the
osl*Semaphore synchronization functions:

- a pthread one (used by default)
- an alternative one based on System V semaphores

The alternative System V implementation is only used if the macro
OSL_USE_SYS_V_SEMAPHORE is defined, which is never the case.

Since it is never used, we may as well remove it. Patch attached.

I thought it would be best to not remove such an important piece of code
without some discussion first.

Na, don't think its very important. :) The corresponding headers were marked as obsolete, anyway, and I don't know of any code in LO using those semaphores off the top of my head.

Pushed now.  Thanks,
Stephan
_______________________________________________
LibreOffice mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to