XQuartz: Fix a bunch of compilation warnings about style
(cherry picked from commit 54000bdcbca52a2de31f7c1a1147de6d8e9dbbb8)
This commit is contained in:
parent
96780eaf32
commit
67a51cd9ef
|
@ -277,7 +277,7 @@ __GLXconfig *__glXAquaCreateVisualConfigs(int *numConfigsPtr, int screenNumber)
|
|||
(c-1)->next = NULL;
|
||||
|
||||
if (c - visualConfigs != numConfigs) {
|
||||
FatalError("numConfigs calculation error in setVisualConfigs! numConfigs is %d i is %d\n", numConfigs, c - visualConfigs);
|
||||
FatalError("numConfigs calculation error in setVisualConfigs! numConfigs is %d i is %d\n", numConfigs, (int)(c - visualConfigs));
|
||||
}
|
||||
|
||||
freeGlCapabilities(&caps);
|
||||
|
|
|
@ -243,23 +243,26 @@ static void message_kit_thread (SEL selector, NSObject *arg) {
|
|||
if(!for_appkit) {
|
||||
NSPoint NSlocation = [e locationInWindow];
|
||||
NSWindow *window = [e window];
|
||||
|
||||
NSRect NSframe, NSvisibleFrame;
|
||||
CGRect CGframe, CGvisibleFrame;
|
||||
CGPoint CGlocation;
|
||||
|
||||
if (window != nil) {
|
||||
NSRect frame = [window frame];
|
||||
NSlocation.x += frame.origin.x;
|
||||
NSlocation.y += frame.origin.y;
|
||||
}
|
||||
|
||||
NSRect NSframe = [[NSScreen mainScreen] frame];
|
||||
NSRect NSvisibleFrame = [[NSScreen mainScreen] visibleFrame];
|
||||
NSframe = [[NSScreen mainScreen] frame];
|
||||
NSvisibleFrame = [[NSScreen mainScreen] visibleFrame];
|
||||
|
||||
CGRect CGframe = CGRectMake(NSframe.origin.x, NSframe.origin.y,
|
||||
CGframe = CGRectMake(NSframe.origin.x, NSframe.origin.y,
|
||||
NSframe.size.width, NSframe.size.height);
|
||||
CGRect CGvisibleFrame = CGRectMake(NSvisibleFrame.origin.x,
|
||||
CGvisibleFrame = CGRectMake(NSvisibleFrame.origin.x,
|
||||
NSvisibleFrame.origin.y,
|
||||
NSvisibleFrame.size.width,
|
||||
NSvisibleFrame.size.height);
|
||||
CGPoint CGlocation = CGPointMake(NSlocation.x, NSlocation.y);
|
||||
CGlocation = CGPointMake(NSlocation.x, NSlocation.y);
|
||||
|
||||
if(CGRectContainsPoint(CGframe, CGlocation) &&
|
||||
!CGRectContainsPoint(CGvisibleFrame, CGlocation))
|
||||
|
@ -334,6 +337,7 @@ static void message_kit_thread (SEL selector, NSObject *arg) {
|
|||
case NSApplicationActivatedEventType:
|
||||
for_x = NO;
|
||||
if ([self modalWindow] == nil) {
|
||||
BOOL switch_on_activate, ok;
|
||||
for_appkit = NO;
|
||||
|
||||
/* FIXME: hack to avoid having to pass the event to appkit,
|
||||
|
@ -344,7 +348,6 @@ static void message_kit_thread (SEL selector, NSObject *arg) {
|
|||
|
||||
/* Get the Spaces preference for SwitchOnActivate */
|
||||
(void)CFPreferencesAppSynchronize(CFSTR(".GlobalPreferences"));
|
||||
BOOL switch_on_activate, ok;
|
||||
switch_on_activate = CFPreferencesGetAppBooleanValue(CFSTR("AppleSpacesSwitchOnActivate"), CFSTR(".GlobalPreferences"), &ok);
|
||||
if(!ok)
|
||||
switch_on_activate = YES;
|
||||
|
|
|
@ -632,6 +632,8 @@ objectValueForTableColumn:(NSTableColumn *)tableColumn row:(int)row
|
|||
|
||||
- (IBAction)prefs_changed:sender
|
||||
{
|
||||
BOOL pbproxy_active;
|
||||
|
||||
darwinFakeButtons = [fake_buttons intValue];
|
||||
quartzUseSysBeep = [use_sysbeep intValue];
|
||||
X11EnableKeyEquivalents = [enable_keyequivs intValue];
|
||||
|
@ -653,7 +655,7 @@ objectValueForTableColumn:(NSTableColumn *)tableColumn row:(int)row
|
|||
[NSApp prefs_set_boolean:@PREFS_NO_TCP value:![enable_tcp intValue]];
|
||||
[NSApp prefs_set_integer:@PREFS_DEPTH value:[depth selectedTag]];
|
||||
|
||||
BOOL pbproxy_active = [sync_pasteboard intValue];
|
||||
pbproxy_active = [sync_pasteboard intValue];
|
||||
|
||||
[NSApp prefs_set_boolean:@PREFS_SYNC_PB value:pbproxy_active];
|
||||
[NSApp prefs_set_boolean:@PREFS_SYNC_PB_TO_CLIPBOARD value:[sync_pasteboard_to_clipboard intValue]];
|
||||
|
|
|
@ -210,10 +210,7 @@ updateEventMask (WMEventPtr *pHead)
|
|||
|
||||
/*ARGSUSED*/
|
||||
static int
|
||||
WMFreeClient (data, id)
|
||||
pointer data;
|
||||
XID id;
|
||||
{
|
||||
WMFreeClient (pointer data, XID id) {
|
||||
WMEventPtr pEvent;
|
||||
WMEventPtr *pHead, pCur, pPrev;
|
||||
|
||||
|
@ -237,10 +234,7 @@ WMFreeClient (data, id)
|
|||
|
||||
/*ARGSUSED*/
|
||||
static int
|
||||
WMFreeEvents (data, id)
|
||||
pointer data;
|
||||
XID id;
|
||||
{
|
||||
WMFreeEvents (pointer data, XID id) {
|
||||
WMEventPtr *pHead, pCur, pNext;
|
||||
|
||||
pHead = (WMEventPtr *) data;
|
||||
|
@ -344,10 +338,7 @@ ProcAppleWMSelectInput (register ClientPtr client)
|
|||
*/
|
||||
|
||||
void
|
||||
AppleWMSendEvent (type, mask, which, arg)
|
||||
int type, which, arg;
|
||||
unsigned int mask;
|
||||
{
|
||||
AppleWMSendEvent (int type, unsigned int mask, int which, int arg) {
|
||||
WMEventPtr *pHead, pEvent;
|
||||
ClientPtr client;
|
||||
xAppleWMNotifyEvent se;
|
||||
|
@ -712,9 +703,7 @@ ProcAppleWMDispatch (
|
|||
}
|
||||
|
||||
static void
|
||||
SNotifyEvent(from, to)
|
||||
xAppleWMNotifyEvent *from, *to;
|
||||
{
|
||||
SNotifyEvent(xAppleWMNotifyEvent *from, xAppleWMNotifyEvent *to) {
|
||||
to->type = from->type;
|
||||
to->kind = from->kind;
|
||||
cpswaps (from->sequenceNumber, to->sequenceNumber);
|
||||
|
|
|
@ -513,13 +513,14 @@ void DarwinSendProximityEvents(int ev_type, float pointer_x, float pointer_y) {
|
|||
void DarwinSendScrollEvents(float count_x, float count_y,
|
||||
float pointer_x, float pointer_y,
|
||||
float pressure, float tilt_x, float tilt_y) {
|
||||
int sign_x, sign_y;
|
||||
if(!darwinEvents) {
|
||||
DEBUG_LOG("DarwinSendScrollEvents called before darwinEvents was initialized\n");
|
||||
return;
|
||||
}
|
||||
|
||||
int sign_x = count_x > 0.0f ? SCROLLWHEELLEFTFAKE : SCROLLWHEELRIGHTFAKE;
|
||||
int sign_y = count_y > 0.0f ? SCROLLWHEELUPFAKE : SCROLLWHEELDOWNFAKE;
|
||||
sign_x = count_x > 0.0f ? SCROLLWHEELLEFTFAKE : SCROLLWHEELRIGHTFAKE;
|
||||
sign_y = count_y > 0.0f ? SCROLLWHEELUPFAKE : SCROLLWHEELDOWNFAKE;
|
||||
count_x = fabs(count_x);
|
||||
count_y = fabs(count_y);
|
||||
|
||||
|
|
|
@ -149,15 +149,17 @@ static int accept_fd_handoff(int connected_fd) {
|
|||
char databuf[] = "display";
|
||||
struct iovec iov[1];
|
||||
|
||||
iov[0].iov_base = databuf;
|
||||
iov[0].iov_len = sizeof(databuf);
|
||||
|
||||
union {
|
||||
struct cmsghdr hdr;
|
||||
char bytes[CMSG_SPACE(sizeof(int))];
|
||||
} buf;
|
||||
|
||||
struct msghdr msg;
|
||||
struct cmsghdr *cmsg;
|
||||
|
||||
iov[0].iov_base = databuf;
|
||||
iov[0].iov_len = sizeof(databuf);
|
||||
|
||||
msg.msg_iov = iov;
|
||||
msg.msg_iovlen = 1;
|
||||
msg.msg_control = buf.bytes;
|
||||
|
@ -166,7 +168,7 @@ static int accept_fd_handoff(int connected_fd) {
|
|||
msg.msg_namelen = 0;
|
||||
msg.msg_flags = 0;
|
||||
|
||||
struct cmsghdr *cmsg = CMSG_FIRSTHDR (&msg);
|
||||
cmsg = CMSG_FIRSTHDR (&msg);
|
||||
cmsg->cmsg_level = SOL_SOCKET;
|
||||
cmsg->cmsg_type = SCM_RIGHTS;
|
||||
cmsg->cmsg_len = CMSG_LEN(sizeof(int));
|
||||
|
@ -197,6 +199,7 @@ static void socket_handoff_thread(void *arg) {
|
|||
socket_handoff_t *handoff_data = (socket_handoff_t *)arg;
|
||||
int launchd_fd = -1;
|
||||
int connected_fd;
|
||||
unsigned remain;
|
||||
|
||||
/* Now actually get the passed file descriptor from this connection
|
||||
* If we encounter an error, keep listening.
|
||||
|
@ -229,7 +232,7 @@ static void socket_handoff_thread(void *arg) {
|
|||
* into it.
|
||||
*/
|
||||
|
||||
unsigned remain = 3000000;
|
||||
remain = 3000000;
|
||||
fprintf(stderr, "X11.app: Received new $DISPLAY fd: %d ... sleeping to allow xinitrc to catchup.\n", launchd_fd);
|
||||
while((remain = usleep(remain)) > 0);
|
||||
|
||||
|
@ -626,11 +629,11 @@ static char *command_from_prefs(const char *key, const char *default_value) {
|
|||
|
||||
if ((PlistRef == NULL) || (CFGetTypeID(PlistRef) != CFStringGetTypeID())) {
|
||||
CFStringRef cfDefaultValue = CFStringCreateWithCString(NULL, default_value, kCFStringEncodingASCII);
|
||||
int len = strlen(default_value) + 1;
|
||||
|
||||
CFPreferencesSetAppValue(cfKey, cfDefaultValue, kCFPreferencesCurrentApplication);
|
||||
CFPreferencesAppSynchronize(kCFPreferencesCurrentApplication);
|
||||
|
||||
int len = strlen(default_value) + 1;
|
||||
command = (char *)malloc(len * sizeof(char));
|
||||
if(!command)
|
||||
return NULL;
|
||||
|
|
|
@ -36,7 +36,7 @@
|
|||
|
||||
#include "launchd_fd.h"
|
||||
|
||||
int launchd_display_fd() {
|
||||
int launchd_display_fd(void) {
|
||||
launch_data_t sockets_dict, checkin_request, checkin_response;
|
||||
launch_data_t listening_fd_array, listening_fd;
|
||||
|
||||
|
|
|
@ -69,13 +69,12 @@ static char x11_path[PATH_MAX + 1];
|
|||
|
||||
static pid_t x11app_pid = 0;
|
||||
|
||||
static void set_x11_path() {
|
||||
static void set_x11_path(void) {
|
||||
#if MAC_OS_X_VERSION_MIN_REQUIRED >= 1050
|
||||
|
||||
CFURLRef appURL = NULL;
|
||||
CFBundleRef bundle = NULL;
|
||||
OSStatus osstatus = LSFindApplicationForInfo(kLSUnknownCreator, CFSTR(kX11AppBundleId), nil, nil, &appURL);
|
||||
UInt32 ver;
|
||||
|
||||
switch (osstatus) {
|
||||
case noErr:
|
||||
|
@ -148,15 +147,17 @@ static void send_fd_handoff(int connected_fd, int launchd_fd) {
|
|||
char databuf[] = "display";
|
||||
struct iovec iov[1];
|
||||
|
||||
iov[0].iov_base = databuf;
|
||||
iov[0].iov_len = sizeof(databuf);
|
||||
|
||||
union {
|
||||
struct cmsghdr hdr;
|
||||
char bytes[CMSG_SPACE(sizeof(int))];
|
||||
} buf;
|
||||
|
||||
struct msghdr msg;
|
||||
struct cmsghdr *cmsg;
|
||||
|
||||
iov[0].iov_base = databuf;
|
||||
iov[0].iov_len = sizeof(databuf);
|
||||
|
||||
msg.msg_iov = iov;
|
||||
msg.msg_iovlen = 1;
|
||||
msg.msg_control = buf.bytes;
|
||||
|
@ -165,7 +166,7 @@ static void send_fd_handoff(int connected_fd, int launchd_fd) {
|
|||
msg.msg_namelen = 0;
|
||||
msg.msg_flags = 0;
|
||||
|
||||
struct cmsghdr *cmsg = CMSG_FIRSTHDR (&msg);
|
||||
cmsg = CMSG_FIRSTHDR (&msg);
|
||||
cmsg->cmsg_level = SOL_SOCKET;
|
||||
cmsg->cmsg_type = SCM_RIGHTS;
|
||||
cmsg->cmsg_len = CMSG_LEN(sizeof(int));
|
||||
|
@ -231,10 +232,11 @@ int main(int argc, char **argv, char **envp) {
|
|||
|
||||
kr = bootstrap_look_up(bootstrap_port, server_bootstrap_name, &mp);
|
||||
if(kr != KERN_SUCCESS) {
|
||||
pid_t child;
|
||||
set_x11_path();
|
||||
|
||||
/* This forking is ugly and will be cleaned up later */
|
||||
pid_t child = fork();
|
||||
child = fork();
|
||||
if(child == -1) {
|
||||
fprintf(stderr, "Xquartz: Could not fork: %s\n", strerror(errno));
|
||||
return EXIT_FAILURE;
|
||||
|
|
|
@ -366,7 +366,7 @@ void QuartzSetRootless(Bool state) {
|
|||
* Calls mode specific screen resume to restore the X clip regions
|
||||
* (if needed) and the X server cursor state.
|
||||
*/
|
||||
void QuartzShow() {
|
||||
void QuartzShow(void) {
|
||||
int i;
|
||||
|
||||
if (quartzServerVisible)
|
||||
|
|
|
@ -130,4 +130,6 @@ void QuartzSpaceChanged(uint32_t space_id);
|
|||
|
||||
void QuartzSetFullscreen(Bool state);
|
||||
void QuartzSetRootless(Bool state);
|
||||
|
||||
int server_main(int argc, char **argv, char **envp);
|
||||
#endif
|
||||
|
|
|
@ -311,8 +311,10 @@ void QuartzAudioInit(void)
|
|||
|
||||
// Prepare for playback
|
||||
#if MAC_OS_X_VERSION_MIN_REQUIRED >= 1050
|
||||
{
|
||||
AudioDeviceIOProcID sInputIOProcID = NULL;
|
||||
status = AudioDeviceCreateIOProcID( outputDevice, QuartzAudioIOProc, &data, &sInputIOProcID );
|
||||
}
|
||||
#else
|
||||
status = AudioDeviceAddIOProc(outputDevice, QuartzAudioIOProc, &data);
|
||||
#endif
|
||||
|
|
|
@ -255,10 +255,10 @@ ProcAppleDRIDestroySurface(
|
|||
register ClientPtr client
|
||||
)
|
||||
{
|
||||
int rc;
|
||||
REQUEST(xAppleDRIDestroySurfaceReq);
|
||||
DrawablePtr pDrawable;
|
||||
REQUEST_SIZE_MATCH(xAppleDRIDestroySurfaceReq);
|
||||
int rc;
|
||||
|
||||
rc = dixLookupDrawable(&pDrawable, stuff->drawable, client, 0,
|
||||
DixReadAccess);
|
||||
|
|
|
@ -72,7 +72,7 @@ X_PFX (cvt_vptr_to_uint) (void * val) __attribute__((always_inline));
|
|||
static __inline__ void *
|
||||
X_PFX (cvt_uint_to_vptr) (unsigned int val)
|
||||
{
|
||||
return (void*)((size_t)(val));
|
||||
return (void*)((unsigned long)(val));
|
||||
}
|
||||
|
||||
static __inline__ unsigned int
|
||||
|
|
|
@ -215,10 +215,10 @@ xprDestroyFrame(RootlessFrameID wid)
|
|||
static void
|
||||
xprMoveFrame(RootlessFrameID wid, ScreenPtr pScreen, int newX, int newY)
|
||||
{
|
||||
TA_SERVER();
|
||||
|
||||
xp_window_changes wc;
|
||||
|
||||
TA_SERVER();
|
||||
|
||||
wc.x = newX;
|
||||
wc.y = newY;
|
||||
// ErrorF("xprMoveFrame(%d, %p, %d, %d)\n", wid, pScreen, newX, newY);
|
||||
|
@ -272,7 +272,7 @@ static void xprRestackFrame(RootlessFrameID wid, RootlessFrameID nextWid) {
|
|||
}
|
||||
|
||||
if(window_hash) {
|
||||
RootlessWindowRec *winRec = x_hash_table_lookup(window_hash, x_cvt_uint_to_vptr((xp_window_id)wid), NULL);
|
||||
RootlessWindowRec *winRec = x_hash_table_lookup(window_hash, wid, NULL);
|
||||
|
||||
if(winRec) {
|
||||
if(quartzEnableRootless)
|
||||
|
@ -566,8 +566,8 @@ xprHideWindows(Bool hide)
|
|||
TA_SERVER();
|
||||
|
||||
for (screen = 0; screen < screenInfo.numScreens; screen++) {
|
||||
pRoot = WindowTable[screenInfo.screens[screen]->myNum];
|
||||
RootlessFrameID prevWid = NULL;
|
||||
pRoot = WindowTable[screenInfo.screens[screen]->myNum];
|
||||
|
||||
for (pWin = pRoot->firstChild; pWin; pWin = pWin->nextSib) {
|
||||
RootlessWindowRec *winRec = WINREC(pWin);
|
||||
|
|
|
@ -317,11 +317,11 @@ xprAddScreen(int index, ScreenPtr pScreen)
|
|||
|
||||
if (noPseudoramiXExtension)
|
||||
{
|
||||
ErrorF("Warning: noPseudoramiXExtension!\n");
|
||||
|
||||
CGDirectDisplayID dpy;
|
||||
CGRect frame;
|
||||
|
||||
ErrorF("Warning: noPseudoramiXExtension!\n");
|
||||
|
||||
dpy = displayAtIndex(index);
|
||||
|
||||
frame = displayScreenBounds(dpy);
|
||||
|
|
Loading…
Reference in New Issue