annotate tests/frame.el @ 5566:4654c01af32b

Improve the implementation, documentation of #'labels, #'flet. lisp/ChangeLog addition: 2011-09-07 Aidan Kehoe <kehoea@parhasard.net> * bytecomp.el: * bytecomp.el (for-effect): Move this earlier in the file, it's referenced in byte-compile-initial-macro-environment. * bytecomp.el (byte-compile-initial-macro-environment): In the byte-compile-macro-environment definition for #'labels, put off the compiling the lambda bodies until the point where the rest of the form is being compiled, allowing the lambda bodies to access appropriate values for byte-compile-bound-variables, and reducing excessive warning about free variables. Add a byte-compile-macro-environment definition for #'flet. This modifies byte-compile-function-environment appropriately, and warns about bindings of functions that have macro definitions in the current environment, about functions that have byte codes, and about functions that have byte-compile methods (which may not do what the user wants at runtime). * bytecomp.el (byte-compile-funcall): If FUNCTION is constant, call #'byte-compile-callargs-warn if that's appropriate, giving warnings about problems with calling functions bound with #'labels. * cl-macs.el: * cl-macs.el (flet): Mention the main difference from Common Lisp, that the bindings are dynamic, not lexical. Counsel the use of #'labels, not #'flet, for this and other reasons. Explain the limited single use case for #'flet. Cross-reference to bytecomp.el in a comment. * cl-macs.el (labels): Go into detail on which functions may be called from where. Explain how to access the function definition of a label within FORM. Add a comment cross-referencing to bytecomp.el. man/ChangeLog addition: 2011-09-07 Aidan Kehoe <kehoea@parhasard.net> * cl.texi (Function Bindings): Move #'labels first, describe it in more detail, explaining that it is to be preferred over #'flet, and explaining why. Explain that dynamic bindings with #'flet will also not work when functions are accessed through their bytecodes.
author Aidan Kehoe <kehoea@parhasard.net>
date Wed, 07 Sep 2011 16:26:45 +0100
parents 308d34e9f07d
children
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
4980
931aaa7a6b37 Martin Buchholz claims tests/frame.el.
Jerry James <james@xemacs.org>
parents: 108
diff changeset
1 ;;; Test geometry setting for frames.
931aaa7a6b37 Martin Buchholz claims tests/frame.el.
Jerry James <james@xemacs.org>
parents: 108
diff changeset
2 ;;;
931aaa7a6b37 Martin Buchholz claims tests/frame.el.
Jerry James <james@xemacs.org>
parents: 108
diff changeset
3 ;;; Copyright (C) 1997 Martin Buchholz
931aaa7a6b37 Martin Buchholz claims tests/frame.el.
Jerry James <james@xemacs.org>
parents: 108
diff changeset
4 ;;;
5402
308d34e9f07d Changed bulk of GPLv2 or later files identified by script
Mats Lidell <matsl@xemacs.org>
parents: 4980
diff changeset
5 ;; This file is part of XEmacs.
308d34e9f07d Changed bulk of GPLv2 or later files identified by script
Mats Lidell <matsl@xemacs.org>
parents: 4980
diff changeset
6 ;;
308d34e9f07d Changed bulk of GPLv2 or later files identified by script
Mats Lidell <matsl@xemacs.org>
parents: 4980
diff changeset
7 ;; XEmacs is free software: you can redistribute it and/or modify it
308d34e9f07d Changed bulk of GPLv2 or later files identified by script
Mats Lidell <matsl@xemacs.org>
parents: 4980
diff changeset
8 ;; under the terms of the GNU General Public License as published by the
308d34e9f07d Changed bulk of GPLv2 or later files identified by script
Mats Lidell <matsl@xemacs.org>
parents: 4980
diff changeset
9 ;; Free Software Foundation, either version 3 of the License, or (at your
308d34e9f07d Changed bulk of GPLv2 or later files identified by script
Mats Lidell <matsl@xemacs.org>
parents: 4980
diff changeset
10 ;; option) any later version.
308d34e9f07d Changed bulk of GPLv2 or later files identified by script
Mats Lidell <matsl@xemacs.org>
parents: 4980
diff changeset
11 ;;
308d34e9f07d Changed bulk of GPLv2 or later files identified by script
Mats Lidell <matsl@xemacs.org>
parents: 4980
diff changeset
12 ;; XEmacs is distributed in the hope that it will be useful, but WITHOUT
308d34e9f07d Changed bulk of GPLv2 or later files identified by script
Mats Lidell <matsl@xemacs.org>
parents: 4980
diff changeset
13 ;; ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
308d34e9f07d Changed bulk of GPLv2 or later files identified by script
Mats Lidell <matsl@xemacs.org>
parents: 4980
diff changeset
14 ;; FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
308d34e9f07d Changed bulk of GPLv2 or later files identified by script
Mats Lidell <matsl@xemacs.org>
parents: 4980
diff changeset
15 ;; for more details.
308d34e9f07d Changed bulk of GPLv2 or later files identified by script
Mats Lidell <matsl@xemacs.org>
parents: 4980
diff changeset
16 ;;
308d34e9f07d Changed bulk of GPLv2 or later files identified by script
Mats Lidell <matsl@xemacs.org>
parents: 4980
diff changeset
17 ;; You should have received a copy of the GNU General Public License
308d34e9f07d Changed bulk of GPLv2 or later files identified by script
Mats Lidell <matsl@xemacs.org>
parents: 4980
diff changeset
18 ;; along with XEmacs. If not, see <http://www.gnu.org/licenses/>.
4980
931aaa7a6b37 Martin Buchholz claims tests/frame.el.
Jerry James <james@xemacs.org>
parents: 108
diff changeset
19
108
360340f9fd5f Import from CVS: tag r20-1b6
cvs
parents:
diff changeset
20 (defmacro check-frame-geometry (xx yy)
360340f9fd5f Import from CVS: tag r20-1b6
cvs
parents:
diff changeset
21 `(loop for frame in (list nil (selected-frame))
360340f9fd5f Import from CVS: tag r20-1b6
cvs
parents:
diff changeset
22 do
360340f9fd5f Import from CVS: tag r20-1b6
cvs
parents:
diff changeset
23 (assert (eq (frame-property frame 'top) ,yy))
360340f9fd5f Import from CVS: tag r20-1b6
cvs
parents:
diff changeset
24 (assert (eq (frame-property frame 'left) ,xx))
360340f9fd5f Import from CVS: tag r20-1b6
cvs
parents:
diff changeset
25 (assert (eq (frame-property frame 'top) ,yy))
360340f9fd5f Import from CVS: tag r20-1b6
cvs
parents:
diff changeset
26 (assert (eq (frame-property frame 'left) ,xx))
360340f9fd5f Import from CVS: tag r20-1b6
cvs
parents:
diff changeset
27 (loop for plist in
360340f9fd5f Import from CVS: tag r20-1b6
cvs
parents:
diff changeset
28 (list (frame-properties)
360340f9fd5f Import from CVS: tag r20-1b6
cvs
parents:
diff changeset
29 (frame-properties nil)
360340f9fd5f Import from CVS: tag r20-1b6
cvs
parents:
diff changeset
30 (frame-properties (selected-frame)))
360340f9fd5f Import from CVS: tag r20-1b6
cvs
parents:
diff changeset
31 do
360340f9fd5f Import from CVS: tag r20-1b6
cvs
parents:
diff changeset
32 (assert (eq (plist-get plist 'top) ,yy))
360340f9fd5f Import from CVS: tag r20-1b6
cvs
parents:
diff changeset
33 (assert (eq (plist-get plist 'left) ,xx)))))
360340f9fd5f Import from CVS: tag r20-1b6
cvs
parents:
diff changeset
34
360340f9fd5f Import from CVS: tag r20-1b6
cvs
parents:
diff changeset
35 (loop for (x y) in '((0 0) (1 1) (3 3) (9 9) (10 20) (20 40) (40 80) (100 200))
360340f9fd5f Import from CVS: tag r20-1b6
cvs
parents:
diff changeset
36 do
360340f9fd5f Import from CVS: tag r20-1b6
cvs
parents:
diff changeset
37 (loop for frame in (list nil (selected-frame))
360340f9fd5f Import from CVS: tag r20-1b6
cvs
parents:
diff changeset
38 do
360340f9fd5f Import from CVS: tag r20-1b6
cvs
parents:
diff changeset
39 (set-frame-properties frame `(left ,x top ,y))
360340f9fd5f Import from CVS: tag r20-1b6
cvs
parents:
diff changeset
40 (check-frame-geometry x y)
360340f9fd5f Import from CVS: tag r20-1b6
cvs
parents:
diff changeset
41 (set-frame-property frame 'top (+ y 3))
360340f9fd5f Import from CVS: tag r20-1b6
cvs
parents:
diff changeset
42 (check-frame-geometry x (+ y 3))
360340f9fd5f Import from CVS: tag r20-1b6
cvs
parents:
diff changeset
43 (set-frame-property frame 'left (+ x 3))
360340f9fd5f Import from CVS: tag r20-1b6
cvs
parents:
diff changeset
44 (check-frame-geometry (+ x 3) (+ y 3))))