Re: Unixware Patch (Was: Re: Beta2 Tag'd and Bundled ...)
От | Larry Rosenman |
---|---|
Тема | Re: Unixware Patch (Was: Re: Beta2 Tag'd and Bundled ...) |
Дата | |
Msg-id | 385900000.1062624200@lerlaptop.iadfw.net обсуждение исходный текст |
Ответ на | Unixware Patch (Was: Re: Beta2 Tag'd and Bundled ...) ("Marc G. Fournier" <scrappy@hub.org>) |
Ответы |
Re: Unixware Patch (Was: Re: Beta2 Tag'd and Bundled ...)
|
Список | pgsql-hackers |
--On Wednesday, September 03, 2003 17:09:49 -0400 Bruce Momjian <pgman@candle.pha.pa.us> wrote: > Larry Rosenman wrote: >> >> >> --On Wednesday, September 03, 2003 16:51:51 -0400 Bruce Momjian >> <pgman@candle.pha.pa.us> wrote: >> >> > Larry Rosenman wrote: >> >> > From UnixWare: >> >> >> >> $ cc -O -Kpthread test_thread.c -o test_thread -lsocket -lnsl >> >> UX:acomp: WARNING: "test_thread.c", line 60: argument #3 incompatible >> >> with prototype: pthread_create() >> >> UX:acomp: WARNING: "test_thread.c", line 61: argument #3 incompatible >> >> with prototype: pthread_create() >> >> $ ./test_thread >> >> Your functions are all thread-safe >> >> $ >> > >> > Well, that's great news, and so clear too! >> > >> > I am curious about the compiler warnings. >> > >> > What does your OS want for the 3rd argument of pthread_create()? I >> > thought a void pointer would be OK for everyone: >> > >> > pthread_create(&thread1, NULL, (void *) func_call_1, NULL); >> >> void *(*start_routine)(void*) >> >> Here is our man page: >> http://lerami.lerctr.org:8458/en/man/html.3pthread/pthread_create.3pthre >> ad. html > > Yes, that's what I have too. What if you have the functions taking > (void *) rather than void. Does that make the warnings disappear? $ r cc cc -O -Kpthread test_thread.c -o test_thread -lsocket -lnsl $ ./test_thread Your functions are all thread-safe $ Attached is my modified version. -- Larry Rosenman http://www.lerctr.org/~ler Phone: +1 972-414-9812 E-Mail: ler@lerctr.org US Mail: 1905 Steamboat Springs Drive, Garland, TX 75044-6749
Вложения
В списке pgsql-hackers по дате отправления: