From d2eea5b594ddfb4f5d99c1dcf710d279ae923a2c Mon Sep 17 00:00:00 2001 From: Paul Eggert Date: Wed, 14 Sep 2011 08:47:21 -0700 Subject: [PATCH] * xselect.c: Use signed conversions more consistently (Bug#9498). MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit (selection_data_to_lisp_data): Assume incoming selection data are signed integers, not unsigned. This is to be consistent with outgoing selection data, which was modified to use signed integers in as part of the fix to Bug#9196 in response to Jan Djärv's comment in that X11 expects long, not unsigned long. --- src/ChangeLog | 10 ++++++++++ src/xselect.c | 8 ++++---- 2 files changed, 14 insertions(+), 4 deletions(-) diff --git a/src/ChangeLog b/src/ChangeLog index 9f8361153a1..fd3de54c3e9 100644 --- a/src/ChangeLog +++ b/src/ChangeLog @@ -1,3 +1,13 @@ +2011-09-14 Paul Eggert + + * xselect.c: Use signed conversions more consistently (Bug#9498). + (selection_data_to_lisp_data): Assume incoming selection data are + signed integers, not unsigned. This is to be consistent with + outgoing selection data, which was modified to use signed integers + in as part of the fix to Bug#9196 in response to Jan D.'s comment + in that X11 + expects long, not unsigned long. + 2011-09-14 Eli Zaretskii * xdisp.c (try_window_reusing_current_matrix): Fix incorrect diff --git a/src/xselect.c b/src/xselect.c index 241622e81b3..29e8552bb9c 100644 --- a/src/xselect.c +++ b/src/xselect.c @@ -1685,9 +1685,9 @@ selection_data_to_lisp_data (Display *display, const unsigned char *data, convert it to a cons of integers, 16 bits in each half. */ else if (format == 32 && size == sizeof (int)) - return INTEGER_TO_CONS (((unsigned int *) data) [0]); + return INTEGER_TO_CONS (((int *) data) [0]); else if (format == 16 && size == sizeof (short)) - return make_number (((unsigned short *) data) [0]); + return make_number (((short *) data) [0]); /* Convert any other kind of data to a vector of numbers, represented as above (as an integer, or a cons of two 16 bit integers.) @@ -1699,7 +1699,7 @@ selection_data_to_lisp_data (Display *display, const unsigned char *data, v = Fmake_vector (make_number (size / 2), make_number (0)); for (i = 0; i < size / 2; i++) { - EMACS_INT j = ((unsigned short *) data) [i]; + EMACS_INT j = ((short *) data) [i]; Faset (v, make_number (i), make_number (j)); } return v; @@ -1711,7 +1711,7 @@ selection_data_to_lisp_data (Display *display, const unsigned char *data, make_number (0)); for (i = 0; i < size / X_LONG_SIZE; i++) { - unsigned int j = ((unsigned int *) data) [i]; + int j = ((int *) data) [i]; Faset (v, make_number (i), INTEGER_TO_CONS (j)); } return v; -- 2.39.2