[PATCH][PUSHED] Remove unused System V oslSemaphore implementation

Stephan Bergmann 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,
Stephan


More information about the LibreOffice mailing list