* select.el (x-set-selection): Doc fix.

(x-valid-simple-selection-p): Disallow selection data consisting
of a list or cons of integers, since that is not used.
(xselect--selection-bounds, xselect--int-to-cons): New functions.
(xselect-convert-to-string, xselect-convert-to-length)
(xselect-convert-to-filename, xselect-convert-to-charpos)
(xselect-convert-to-lineno, xselect-convert-to-colno): Use them.
This commit is contained in:
Chong Yidong 2009-07-14 16:58:25 +00:00
parent 72d36834fc
commit f470187ffb
2 changed files with 89 additions and 137 deletions

View file

@ -1,3 +1,13 @@
2009-07-14 Chong Yidong <cyd@stupidchicken.com>
* select.el (x-set-selection): Doc fix.
(x-valid-simple-selection-p): Disallow selection data consisting
of a list or cons of integers, since that is not used.
(xselect--selection-bounds, xselect--int-to-cons): New functions.
(xselect-convert-to-string, xselect-convert-to-length)
(xselect-convert-to-filename, xselect-convert-to-charpos)
(xselect-convert-to-lineno, xselect-convert-to-colno): Use them.
2009-07-14 Dmitry Dzhus <dima@sphinx.net.ru>
* progmodes/gdb-mi.el (json-partial-output): Fix broken GDB/MI

View file

@ -116,19 +116,21 @@ in `selection-converter-alist', which see."
(defun x-set-selection (type data)
"Make an X Windows selection of type TYPE and value DATA.
The argument TYPE (nil means `PRIMARY') says which selection, and
DATA specifies the contents. TYPE must be a symbol. \(It can also
be a string, which stands for the symbol with that name, but this
is considered obsolete.) DATA may be a string, a symbol, an
integer (or a cons of two integers or list of two integers).
TYPE is a symbol specifying the selection type. This is normally
one of `PRIMARY', `SECONDARY', or `CLIPBOARD'; or nil, which is
equivalent to `PRIMARY'. (It can also be a string, which stands
for the symbol with that name, but this usage is obsolete.)
The selection may also be a cons of two markers pointing to the same buffer,
or an overlay. In these cases, the selection is considered to be the text
between the markers *at whatever time the selection is examined*.
Thus, editing done in the buffer after you specify the selection
can alter the effective value of the selection.
The data may also be a vector of valid non-vector selection values.
DATA is a selection value. It should be one of the following:
- a vector of non-vector selection values
- a string
- an integer
- a cons cell of two markers pointing to the same buffer
- an overlay
In the latter two cases, the selection is considered to be the
text between the markers at whatever time the selection is
examined. Thus, editing done in the buffer after you specify the
selection can alter the effective value of the selection.
The return value is DATA.
@ -138,9 +140,7 @@ prefix argument, it uses the text of the region as the selection value ."
(interactive (if (not current-prefix-arg)
(list 'PRIMARY (read-string "Set text for pasting: "))
(list 'PRIMARY (buffer-substring (region-beginning) (region-end)))))
;; This is for temporary compatibility with pre-release Emacs 19.
(if (stringp type)
(setq type (intern type)))
(if (stringp type) (setq type (intern type)))
(or (x-valid-simple-selection-p data)
(and (vectorp data)
(let ((valid t)
@ -158,24 +158,19 @@ prefix argument, it uses the text of the region as the selection value ."
data)
(defun x-valid-simple-selection-p (data)
(or (stringp data)
(symbolp data)
(integerp data)
(and (consp data)
(integerp (car data))
(or (integerp (cdr data))
(and (consp (cdr data))
(integerp (car (cdr data))))))
(overlayp data)
(and (consp data)
(or (and (consp data)
(markerp (car data))
(markerp (cdr data))
(marker-buffer (car data))
(marker-buffer (cdr data))
(eq (marker-buffer (car data))
(marker-buffer (cdr data)))
(buffer-name (marker-buffer (car data)))
(buffer-name (marker-buffer (cdr data))))))
(eq (marker-buffer (car data))
(marker-buffer (cdr data))))
(stringp data)
(and (overlayp data)
(overlay-buffer data)
(buffer-name (overlay-buffer data)))
(symbolp data)
(integerp data)))
;;; Cut Buffer support
@ -211,31 +206,38 @@ Cut buffers are considered obsolete; you should use selections instead."
;; Every selection type that Emacs handles is implemented this way, except
;; for TIMESTAMP, which is a special case.
(defun xselect--selection-bounds (value)
"Return bounds of X selection value VALUE.
The return value is a list (BEG END BUF) if VALUE is a cons of
two markers or an overlay. Otherwise, it is nil."
(cond ((and (consp value)
(markerp (car value))
(markerp (cdr value)))
(when (and (marker-buffer (car value))
(buffer-name (marker-buffer (car value)))
(eq (marker-buffer (car value))
(marker-buffer (cdr value))))
(list (marker-position (car value))
(marker-position (cdr value))
(marker-buffer (car value)))))
((overlayp value)
(when (overlay-buffer value)
(list (overlay-start value)
(overlay-end value)
(overlay-buffer value))))))
(defun xselect--int-to-cons (n)
(cons (ash n -16) (logand n 65535)))
(defun xselect-convert-to-string (selection type value)
(let (str coding)
;; Get the actual string from VALUE.
(cond ((stringp value)
(setq str value))
((overlayp value)
(save-excursion
(or (buffer-name (overlay-buffer value))
(error "selection is in a killed buffer"))
(set-buffer (overlay-buffer value))
(setq str (buffer-substring (overlay-start value)
(overlay-end value)))))
((and (consp value)
(markerp (car value))
(markerp (cdr value)))
(or (eq (marker-buffer (car value)) (marker-buffer (cdr value)))
(signal 'error
(list "markers must be in the same buffer"
(car value) (cdr value))))
(save-excursion
(set-buffer (or (marker-buffer (car value))
(error "selection is in a killed buffer")))
(setq str (buffer-substring (car value) (cdr value))))))
((setq value (xselect--selection-bounds value))
(with-current-buffer (nth 2 value)
(setq str (buffer-substring (nth 0 value)
(nth 1 value))))))
(when str
;; If TYPE is nil, this is a local request, thus return STR as
;; is. Otherwise, encode STR.
@ -288,31 +290,18 @@ Cut buffers are considered obsolete; you should use selections instead."
(setq str (string-make-unibyte str)))
(t
(error "Unknow selection type: %S" type))
)))
(error "Unknown selection type: %S" type)))))
(setq next-selection-coding-system nil)
(cons type str))))
(defun xselect-convert-to-length (selection type value)
(let ((value
(cond ((stringp value)
(length value))
((overlayp value)
(abs (- (overlay-end value) (overlay-start value))))
((and (consp value)
(markerp (car value))
(markerp (cdr value)))
(or (eq (marker-buffer (car value))
(marker-buffer (cdr value)))
(signal 'error
(list "markers must be in the same buffer"
(car value) (cdr value))))
(abs (- (car value) (cdr value)))))))
(if value ; force it to be in 32-bit format.
(cons (ash value -16) (logand value 65535))
nil)))
(let ((len (cond ((stringp value)
(length value))
((setq value (xselect--selection-bounds value))
(abs (- (nth 0 value) (nth 1 value)))))))
(if len
(xselect--int-to-cons len))))
(defun xselect-convert-to-targets (selection type value)
;; return a vector of atoms, but remove duplicates first.
@ -335,77 +324,31 @@ Cut buffers are considered obsolete; you should use selections instead."
'NULL)
(defun xselect-convert-to-filename (selection type value)
(cond ((overlayp value)
(buffer-file-name (or (overlay-buffer value)
(error "selection is in a killed buffer"))))
((and (consp value)
(markerp (car value))
(markerp (cdr value)))
(buffer-file-name (or (marker-buffer (car value))
(error "selection is in a killed buffer"))))
(t nil)))
(when (setq value (xselect--selection-bounds value))
(buffer-file-name (nth 2 value))))
(defun xselect-convert-to-charpos (selection type value)
(let (a b tmp)
(cond ((cond ((overlayp value)
(setq a (overlay-start value)
b (overlay-end value)))
((and (consp value)
(markerp (car value))
(markerp (cdr value)))
(setq a (car value)
b (cdr value))))
(setq a (1- a) b (1- b)) ; zero-based
(if (< b a) (setq tmp a a b b tmp))
(cons 'SPAN
(vector (cons (ash a -16) (logand a 65535))
(cons (ash b -16) (logand b 65535))))))))
(when (setq value (xselect--selection-bounds value))
(let ((beg (1- (nth 0 value))) ; zero-based
(end (1- (nth 1 value))))
(cons 'SPAN (vector (xselect--int-to-cons (min beg end))
(xselect--int-to-cons (max beg end)))))))
(defun xselect-convert-to-lineno (selection type value)
(let (a b buf tmp)
(cond ((cond ((and (consp value)
(markerp (car value))
(markerp (cdr value)))
(setq a (marker-position (car value))
b (marker-position (cdr value))
buf (marker-buffer (car value))))
((overlayp value)
(setq buf (overlay-buffer value)
a (overlay-start value)
b (overlay-end value)))
)
(save-excursion
(set-buffer buf)
(setq a (count-lines 1 a)
b (count-lines 1 b)))
(if (< b a) (setq tmp a a b b tmp))
(cons 'SPAN
(vector (cons (ash a -16) (logand a 65535))
(cons (ash b -16) (logand b 65535))))))))
(when (setq value (xselect--selection-bounds value))
(with-current-buffer (nth 2 value)
(let ((beg (line-number-at-pos (nth 0 value)))
(end (line-number-at-pos (nth 1 value))))
(cons 'SPAN (vector (xselect--int-to-cons (min beg end))
(xselect--int-to-cons (max beg end))))))))
(defun xselect-convert-to-colno (selection type value)
(let (a b buf tmp)
(cond ((cond ((and (consp value)
(markerp (car value))
(markerp (cdr value)))
(setq a (car value)
b (cdr value)
buf (marker-buffer a)))
((overlayp value)
(setq buf (overlay-buffer value)
a (overlay-start value)
b (overlay-end value)))
)
(save-excursion
(set-buffer buf)
(goto-char a)
(setq a (current-column))
(goto-char b)
(setq b (current-column)))
(if (< b a) (setq tmp a a b b tmp))
(cons 'SPAN
(vector (cons (ash a -16) (logand a 65535))
(cons (ash b -16) (logand b 65535))))))))
(when (setq value (xselect--selection-bounds value))
(with-current-buffer (nth 2 value)
(let ((beg (progn (goto-char (nth 0 value)) (current-column)))
(end (progn (goto-char (nth 1 value)) (current-column))))
(cons 'SPAN (vector (xselect--int-to-cons (min beg end))
(xselect--int-to-cons (max beg end))))))))
(defun xselect-convert-to-os (selection type size)
(symbol-name system-type))
@ -430,7 +373,7 @@ This function returns the string \"emacs\"."
(defun xselect-convert-to-integer (selection type value)
(and (integerp value)
(cons (ash value -16) (logand value 65535))))
(xselect--int-to-cons value)))
(defun xselect-convert-to-atom (selection type value)
(and (symbolp value) value))
@ -457,8 +400,7 @@ This function returns the string \"emacs\"."
(NAME . xselect-convert-to-name)
(ATOM . xselect-convert-to-atom)
(INTEGER . xselect-convert-to-integer)
(_EMACS_INTERNAL . xselect-convert-to-identity)
))
(_EMACS_INTERNAL . xselect-convert-to-identity)))
(provide 'select)