diff options
| author | Paul Eggert | 2011-05-27 12:32:01 -0700 |
|---|---|---|
| committer | Paul Eggert | 2011-05-27 12:32:01 -0700 |
| commit | fb1ac845caea7da6ba98b93c3d67fa67c651b8ef (patch) | |
| tree | 907140b4154fe861d7fa27aeaaf19c7a51934220 /src | |
| parent | f1b54466e9f3b0d0037f161157a8b27dd0fb283f (diff) | |
| download | emacs-fb1ac845caea7da6ba98b93c3d67fa67c651b8ef.tar.gz emacs-fb1ac845caea7da6ba98b93c3d67fa67c651b8ef.zip | |
* xselect.c: Fix minor problems prompted by GCC 4.6.0 warnings.
(x_handle_selection_request, frame_for_x_selection): Remove unused vars.
(x_clipboard_manager_save): Now static.
(Fx_clipboard_manager_save): Rename local to avoid shadowing.
Diffstat (limited to 'src')
| -rw-r--r-- | src/ChangeLog | 5 | ||||
| -rw-r--r-- | src/xselect.c | 13 |
2 files changed, 11 insertions, 7 deletions
diff --git a/src/ChangeLog b/src/ChangeLog index 2373dd81ad1..1ffeba88607 100644 --- a/src/ChangeLog +++ b/src/ChangeLog | |||
| @@ -1,5 +1,10 @@ | |||
| 1 | 2011-05-27 Paul Eggert <eggert@cs.ucla.edu> | 1 | 2011-05-27 Paul Eggert <eggert@cs.ucla.edu> |
| 2 | 2 | ||
| 3 | * xselect.c: Fix minor problems prompted by GCC 4.6.0 warnings. | ||
| 4 | (x_handle_selection_request, frame_for_x_selection): Remove unused vars. | ||
| 5 | (x_clipboard_manager_save): Now static. | ||
| 6 | (Fx_clipboard_manager_save): Rename local to avoid shadowing. | ||
| 7 | |||
| 3 | * fns.c: Fix minor problems prompted by GCC 4.6.0 warnings. | 8 | * fns.c: Fix minor problems prompted by GCC 4.6.0 warnings. |
| 4 | (crypto_hash_function): Now static. | 9 | (crypto_hash_function): Now static. |
| 5 | Fix pointer signedness problems. Avoid unnecessary initializations. | 10 | Fix pointer signedness problems. Avoid unnecessary initializations. |
diff --git a/src/xselect.c b/src/xselect.c index f0ffbe0eb2d..c33d011dba8 100644 --- a/src/xselect.c +++ b/src/xselect.c | |||
| @@ -822,7 +822,6 @@ x_handle_selection_request (struct input_event *event) | |||
| 822 | /* Perform conversions. This can signal. */ | 822 | /* Perform conversions. This can signal. */ |
| 823 | for (j = 0; j < nselections; j++) | 823 | for (j = 0; j < nselections; j++) |
| 824 | { | 824 | { |
| 825 | struct selection_data *cs = converted_selections + j; | ||
| 826 | Lisp_Object subtarget = AREF (multprop, 2*j); | 825 | Lisp_Object subtarget = AREF (multprop, 2*j); |
| 827 | Atom subproperty = symbol_to_x_atom (dpyinfo, | 826 | Atom subproperty = symbol_to_x_atom (dpyinfo, |
| 828 | AREF (multprop, 2*j+1)); | 827 | AREF (multprop, 2*j+1)); |
| @@ -1878,7 +1877,7 @@ x_handle_selection_notify (XSelectionEvent *event) | |||
| 1878 | static struct frame * | 1877 | static struct frame * |
| 1879 | frame_for_x_selection (Lisp_Object object) | 1878 | frame_for_x_selection (Lisp_Object object) |
| 1880 | { | 1879 | { |
| 1881 | Lisp_Object tail, frame; | 1880 | Lisp_Object tail; |
| 1882 | struct frame *f; | 1881 | struct frame *f; |
| 1883 | 1882 | ||
| 1884 | if (NILP (object)) | 1883 | if (NILP (object)) |
| @@ -2110,7 +2109,7 @@ frame's display, or the first available X display. */) | |||
| 2110 | UTF8_STRING property, as described by | 2109 | UTF8_STRING property, as described by |
| 2111 | http://www.freedesktop.org/wiki/ClipboardManager */ | 2110 | http://www.freedesktop.org/wiki/ClipboardManager */ |
| 2112 | 2111 | ||
| 2113 | void | 2112 | static void |
| 2114 | x_clipboard_manager_save (struct x_display_info *dpyinfo, | 2113 | x_clipboard_manager_save (struct x_display_info *dpyinfo, |
| 2115 | Lisp_Object frame) | 2114 | Lisp_Object frame) |
| 2116 | { | 2115 | { |
| @@ -2157,7 +2156,7 @@ FRAME is nil, save all clipboard contents owned by Emacs. */) | |||
| 2157 | { | 2156 | { |
| 2158 | /* Loop through all X displays, saving owned clipboards. */ | 2157 | /* Loop through all X displays, saving owned clipboards. */ |
| 2159 | struct x_display_info *dpyinfo; | 2158 | struct x_display_info *dpyinfo; |
| 2160 | Lisp_Object local_selection, frame; | 2159 | Lisp_Object local_selection, local_frame; |
| 2161 | for (dpyinfo = x_display_list; dpyinfo; dpyinfo = dpyinfo->next) | 2160 | for (dpyinfo = x_display_list; dpyinfo; dpyinfo = dpyinfo->next) |
| 2162 | { | 2161 | { |
| 2163 | local_selection = LOCAL_SELECTION (QCLIPBOARD, dpyinfo); | 2162 | local_selection = LOCAL_SELECTION (QCLIPBOARD, dpyinfo); |
| @@ -2166,9 +2165,9 @@ FRAME is nil, save all clipboard contents owned by Emacs. */) | |||
| 2166 | dpyinfo->Xatom_CLIPBOARD_MANAGER)) | 2165 | dpyinfo->Xatom_CLIPBOARD_MANAGER)) |
| 2167 | continue; | 2166 | continue; |
| 2168 | 2167 | ||
| 2169 | frame = XCAR (XCDR (XCDR (XCDR (local_selection)))); | 2168 | local_frame = XCAR (XCDR (XCDR (XCDR (local_selection)))); |
| 2170 | if (FRAME_LIVE_P (XFRAME (frame))) | 2169 | if (FRAME_LIVE_P (XFRAME (local_frame))) |
| 2171 | x_clipboard_manager_save (dpyinfo, frame); | 2170 | x_clipboard_manager_save (dpyinfo, local_frame); |
| 2172 | } | 2171 | } |
| 2173 | } | 2172 | } |
| 2174 | 2173 | ||