diff options
| author | Dan Nicolaescu | 2010-04-23 00:00:38 -0700 |
|---|---|---|
| committer | Dan Nicolaescu | 2010-04-23 00:00:38 -0700 |
| commit | feeb1604f50bafb987b4e85ac3df5edef5151662 (patch) | |
| tree | fec87f88e822ecec11784c639923d31b241f0b95 /src | |
| parent | d8004abee409ddbda6875bdd3e99bbf1a96e8b9d (diff) | |
| download | emacs-feeb1604f50bafb987b4e85ac3df5edef5151662.tar.gz emacs-feeb1604f50bafb987b4e85ac3df5edef5151662.zip | |
Clean up Solaris code.
* s/sol2-6.h (LD_SWITCH_SYSTEM_TEMACS, C_SWITCH_X_SYSTEM)
(LIB_MOTIF): Remove, configure takes care of this.
(NOT_USING_MOTIF): Remove, unused.
* xrdb.c: Remove #if 0-ed #include.
(SYSV): Remove conditional for old SysV.
* sysdep.c (closedir): Remove conditional code for Solaris,
Solaris has closedir.
Diffstat (limited to 'src')
| -rw-r--r-- | src/ChangeLog | 11 | ||||
| -rw-r--r-- | src/s/sol2-6.h | 29 | ||||
| -rw-r--r-- | src/sysdep.c | 7 | ||||
| -rw-r--r-- | src/xrdb.c | 16 |
4 files changed, 11 insertions, 52 deletions
diff --git a/src/ChangeLog b/src/ChangeLog index 6588549e496..6797b623db2 100644 --- a/src/ChangeLog +++ b/src/ChangeLog | |||
| @@ -1,3 +1,14 @@ | |||
| 1 | 2010-04-23 Dan Nicolaescu <dann@ics.uci.edu> | ||
| 2 | |||
| 3 | Clean up Solaris code. | ||
| 4 | * s/sol2-6.h (LD_SWITCH_SYSTEM_TEMACS, C_SWITCH_X_SYSTEM) | ||
| 5 | (LIB_MOTIF): Remove, configure takes care of this. | ||
| 6 | (NOT_USING_MOTIF): Remove, unused. | ||
| 7 | * xrdb.c: Remove #if 0-ed #include. | ||
| 8 | (SYSV): Remove conditional for old SysV. | ||
| 9 | * sysdep.c (closedir): Remove conditional code for Solaris, | ||
| 10 | Solaris has closedir. | ||
| 11 | |||
| 1 | 2010-04-22 Jan Djärv <jan.h.d@swipnet.se> | 12 | 2010-04-22 Jan Djärv <jan.h.d@swipnet.se> |
| 2 | 13 | ||
| 3 | * xsettings.c (read_and_apply_settings): Check if current_font is | 14 | * xsettings.c (read_and_apply_settings): Check if current_font is |
diff --git a/src/s/sol2-6.h b/src/s/sol2-6.h index 408a0a2ed02..5ca5cfbc0d0 100644 --- a/src/s/sol2-6.h +++ b/src/s/sol2-6.h | |||
| @@ -73,35 +73,6 @@ along with GNU Emacs. If not, see <http://www.gnu.org/licenses/>. */ | |||
| 73 | pty_name[sizeof (pty_name) - 1] = 0; \ | 73 | pty_name[sizeof (pty_name) - 1] = 0; \ |
| 74 | } | 74 | } |
| 75 | 75 | ||
| 76 | /* `#ifdef USE_MOTIF' won't work here, since USE_MOTIF isn't defined yet. | ||
| 77 | Instead, dynamically check whether USE_MOTIF expands to something. */ | ||
| 78 | #define NOT_USING_MOTIF { set x USE_MOTIF; test "$$2" = "USE_MOTIF"; } | ||
| 79 | |||
| 80 | #ifndef __GNUC__ | ||
| 81 | #define LD_SWITCH_SYSTEM_TEMACS -L/usr/ccs/lib LD_SWITCH_X_SITE_AUX \ | ||
| 82 | `NOT_USING_MOTIF || echo ' -R/usr/dt/lib'` | ||
| 83 | #else /* GCC */ | ||
| 84 | /* We use ./prefix-args because we don't know whether LD_SWITCH_X_SITE_AUX | ||
| 85 | has anything in it. It can be empty. | ||
| 86 | This works ok in temacs. */ | ||
| 87 | #define LD_SWITCH_SYSTEM_TEMACS -L/usr/ccs/lib \ | ||
| 88 | `./prefix-args -Xlinker LD_SWITCH_X_SITE_AUX` \ | ||
| 89 | `NOT_USING_MOTIF || echo ' -R/usr/dt/lib -L/usr/dt/lib'` | ||
| 90 | |||
| 91 | /* Get rid of -traditional and let const really do its thing. */ | ||
| 92 | #undef C_SWITCH_SYSTEM | ||
| 93 | #undef const | ||
| 94 | #endif /* GCC */ | ||
| 95 | |||
| 96 | /* Gregory Neil Shapiro <gshapiro@hhmi.org> reports the Motif header files | ||
| 97 | are in this directory on Solaris 2.4. */ | ||
| 98 | #define C_SWITCH_X_SYSTEM -I/usr/dt/include | ||
| 99 | |||
| 100 | /* -lgen is needed for the regex and regcmp functions | ||
| 101 | which are used by Motif. In the future we can try changing | ||
| 102 | regex.c to provide them in Emacs, but this is safer for now. */ | ||
| 103 | #define LIB_MOTIF -lXm -lgen | ||
| 104 | |||
| 105 | /* This is the only known way to avoid some crashes | 76 | /* This is the only known way to avoid some crashes |
| 106 | that seem to relate to screwed up malloc data | 77 | that seem to relate to screwed up malloc data |
| 107 | after deleting a frame. */ | 78 | after deleting a frame. */ |
diff --git a/src/sysdep.c b/src/sysdep.c index 506af23ef30..aebf6e7a2d1 100644 --- a/src/sysdep.c +++ b/src/sysdep.c | |||
| @@ -2709,13 +2709,6 @@ closedir (DIR *dirp /* stream from opendir */) | |||
| 2709 | int rtnval; | 2709 | int rtnval; |
| 2710 | 2710 | ||
| 2711 | rtnval = emacs_close (dirp->dd_fd); | 2711 | rtnval = emacs_close (dirp->dd_fd); |
| 2712 | |||
| 2713 | /* Some systems (like Solaris) allocate the buffer and the DIR all | ||
| 2714 | in one block. Why in the world are we freeing this ourselves | ||
| 2715 | anyway? */ | ||
| 2716 | #if ! defined (SOLARIS2) | ||
| 2717 | xfree ((char *) dirp->dd_buf); /* directory block defined in <dirent.h> */ | ||
| 2718 | #endif | ||
| 2719 | xfree ((char *) dirp); | 2712 | xfree ((char *) dirp); |
| 2720 | 2713 | ||
| 2721 | return rtnval; | 2714 | return rtnval; |
diff --git a/src/xrdb.c b/src/xrdb.c index 7808bacc59f..bd3474c0466 100644 --- a/src/xrdb.c +++ b/src/xrdb.c | |||
| @@ -34,24 +34,8 @@ along with GNU Emacs. If not, see <http://www.gnu.org/licenses/>. */ | |||
| 34 | #include <stdio.h> | 34 | #include <stdio.h> |
| 35 | #include <setjmp.h> | 35 | #include <setjmp.h> |
| 36 | 36 | ||
| 37 | #if 1 /* I'd really appreciate it if this code could go away... -JimB */ | ||
| 38 | /* This avoids lossage in the `dual-universe' headers on AT&T SysV | ||
| 39 | X11. Don't do it on Solaris, because it breaks compilation with | ||
| 40 | XFree86 4.0.3 (and probably many other X11R6 releases) on Solaris | ||
| 41 | 2 */ | ||
| 42 | #if defined(USG5) && !defined(SOLARIS2) | ||
| 43 | #ifndef SYSV | ||
| 44 | #define SYSV | ||
| 45 | #endif | ||
| 46 | #endif /* USG5 && !SOLARIS2 */ | ||
| 47 | |||
| 48 | #endif /* 1 */ | ||
| 49 | |||
| 50 | #include <X11/Xlib.h> | 37 | #include <X11/Xlib.h> |
| 51 | #include <X11/Xatom.h> | 38 | #include <X11/Xatom.h> |
| 52 | #if 0 | ||
| 53 | #include <X11/Xos.h> | ||
| 54 | #endif | ||
| 55 | #include <X11/X.h> | 39 | #include <X11/X.h> |
| 56 | #include <X11/Xutil.h> | 40 | #include <X11/Xutil.h> |
| 57 | #include <X11/Xresource.h> | 41 | #include <X11/Xresource.h> |