--- Begin Message ---
Subject: |
24.5; Commit 35cd5181450 breaks nsterm.m on Mac OS X 10.6 |
Date: |
Sun, 8 Nov 2015 05:16:07 -0800 |
User-agent: |
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.11; rv:42.0) Gecko/20100101 Thunderbird/42.0 |
I'm getting this when trying to build on Mac OS X 10.6:
CC nsterm.o
nsterm.m: In function '-[EmacsView toggleFullScreen:]':
nsterm.m:7058: error: 'NSWindowWillEnterFullScreenNotification'
undeclared (first use in this function)
nsterm.m:7058: error: (Each undeclared identifier is reported only once
nsterm.m:7058: error: for each function it appears in.)
nsterm.m:7066: error: 'NSWindowDidEnterFullScreenNotification'
undeclared (first use in this function)
nsterm.m:7098: error: 'NSWindowWillExitFullScreenNotification'
undeclared (first use in this function)
nsterm.m:7104: error: 'NSWindowDidExitFullScreenNotification'
undeclared (first use in this function)
make[1]: *** [nsterm.o] Error 1
make: *** [src] Error 2
make failed
Looks like this commit is the culprit:
commit 35cd51814507987b916c4b4e0a7b45e09e454341
Author: David Reitter <address@hidden>
Date: Sat Nov 7 20:43:29 2015 -0500
Provide NS notification objects where required to eliminate warnings
* nsterm.m (windowDidResize:, toggleFullScreen:):
Call notification functions with notification objects
as per delegate APIs.
NSWindow.h says:
APPKIT_EXTERN NSString * const NSWindowWillEnterFullScreenNotification
NS_AVAILABLE_MAC(10_7);
APPKIT_EXTERN NSString * const NSWindowDidEnterFullScreenNotification
NS_AVAILABLE_MAC(10_7);
APPKIT_EXTERN NSString * const NSWindowWillExitFullScreenNotification
NS_AVAILABLE_MAC(10_7);
APPKIT_EXTERN NSString * const NSWindowDidExitFullScreenNotification
NS_AVAILABLE_MAC(10_7);
We could fix this by wrapping NSWindow*FullScreenNotification with
preprocessor guards. On the other hand, that's getting awfully bulky for
creating a bunch of NSNotifications that aren't ever used (follow the
function calls).
Perhaps we can fix the warning a different way?
-David
smime.p7s
Description: S/MIME Cryptographic Signature
--- End Message ---
--- Begin Message ---
Subject: |
Re: bug#21862: Acknowledgement (24.5; Commit 35cd5181450 breaks nsterm.m on Mac OS X 10.6) |
Date: |
Tue, 10 Nov 2015 16:02:02 -0500 |
Hi David (and Anders),
I have checked in some changes that should allow compilation on 10.6.8.
I do not have a machine to test, so please reopen this bug in case you
encounter a problem.
- David
> On Nov 10, 2015, at 2:37 PM, David Caldwell <address@hidden> wrote:
>
> Hi David, I'm pulling you in to this bug report since it is about your
> commit.
>
> I tried reverting the commit in question
> (35cd51814507987b916c4b4e0a7b45e09e454341) and I don't see any warning
> generated with the latest XCode (Version 7.1.1 (7B1005)). Where were you
> seeing these warnings?
>
> -David
>
>
--- End Message ---