This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch]: Merge from gnustep for obj-c. Remove warnings for target w64.


"Andrew Pinski" <pinskia@gmail.com> wrote on 19.11.2008 19:23:16:

> objc_thread_exit should have been marked as noreturn.  Do you know why
> it was not?

Well, this is related to __objc_thread_exit, which is prototyped to return 
a value. So I guess there are targets for which this function can return 
?!
So I think the abort () hack should work here and is the best way to solve 
this issue.


> With the above changes and the undef change this is OK.

2008-11-20  Kai Tietz  <kai.tietz@onevision.com>

        *  Object.m (errno): Replaced by errno.h include.
        (compare): Cast self to id to prevent warning on comparison.
        * objc/objc.h (BOOL): Prevent redeclaration of BOOL, if it is 
already there.
        * sendmsg.c (__objc_print_dtable_stats): Remove type warnings.
        * thr-win32.c (__objc_thread_detach): Remove type warning.
        (__objc_thread_id): Likewise.
        * thr.c (__objc_thread_detach_functiont): Add abort () for 
noreturn.



Ok, for apply?

Cheers,
Kai Tietz

|  (\_/)  This is Bunny. Copy and paste Bunny
| (='.'=) into your signature to help him gain
| (")_(") world domination.

Attachment: libobjc.diff
Description: Binary data


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]