svn commit: r268137 - head/sys/sys

[David Chisnall]

On 19 Jun 2015, at 11:45, Hans Petter Selasky <hps at selasky.org> wrote:
> 
> Appearently this will be fixed in GNUSTEP base:
> 
> http://cvsweb.netbsd.org/bsdweb.cgi/src/sys/sys/cdefs_elf.h?only_with_tag=MAIN
> 
> Is this still an issue?

It is impossible to fix it in GNUstep Base, because we can’t guarantee that user code doesn’t include system headers after including GNUstep headers (not to mention the fact that GNUstep is not the only Objective-C standard library implementation out there).  If the user does, for example:

#import <Foundation/Foundation.h>
#include <something_that_includes_cdefs.h>

void example()
{
	__weak id foo = bar();
	baz(foo);
}

Then they will get a compile error no matter what GNUstep’s Foundation.h does.  It can’t prevent cdefs.h from redefining __weak to be something different.

I’ve just looked at the GNUstep base changelog since that NetBSD commit and there are no relevant changes, so I’ve no idea what the NetBSD people are thinking there.

David