[FFmpeg-devel] [PATCH] Reimplementation of OS/2 threads model
Thu Apr 29 14:45:05 CEST 2010
KO Myung-Hun <komh at chollian.net> writes:
> M?ns Rullg?rd wrote:
>> Ramiro Polla <ramiro.polla at gmail.com> writes:
>>> On Wed, Apr 28, 2010 at 11:45 AM, KO Myung-Hun <komh at chollian.net> wrote:
>>>> But now pthreads codes are in os2thread.c, so pthread lib is
>>>> unnecessary. And we can use all the codes of pthread.c without modifying
>>>> it at all. If possible, I would like to use '#include "pthread.c"' using
>>>> some guard macros. This reduces the maintenance burden you are saying.
>>> Could you not make that code a simple library with a subset of
>>> pthreads that ffmpeg needs? You could then link against that library
>>> with the current pthreads code in FFmpeg.
>> But a perfectly good pthread library _already exists_. Why can't we
>> just link to that instead of duplicating the effort.
> I don't understand correctly what you mean by 'perfectly good'. It is
> also a subset of pthreads to support MySQL for OS/2.
If the existing one is missing something, improve it rather than
creating another half-arsed version.
> And we can get two gains at least. One is a less size. The other is the
> less dependency.
Depending on typical system libraries is perfectly OK. Duplicating
them is not.
> Nevertheless, why do you think that this is just the duplication ?
Because it is. You are duplicating the functionality of the existing
pthread library. That's silly.
mans at mansr.com
More information about the ffmpeg-devel