[PATCH][PUSHED] Remove unused System V oslSemaphore implementation
sbergman at redhat.com
Mon Feb 6 01:46:48 PST 2012
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,
More information about the LibreOffice