test: `visible-buffer-substring' should not return text properties.
authorDavid Edmondson <dme@dme.org>
Thu, 26 Jan 2012 07:19:37 +0000 (07:19 +0000)
committerDavid Bremner <bremner@debian.org>
Thu, 26 Jan 2012 12:35:02 +0000 (08:35 -0400)
When using `visible-buffer-substring' to examine a buffer, the text
properties are not useful, so don't include them.

test/test-lib.el

index bc75f06e3ab8294e39416c73acd5e84a79823892..5b32e0abe4d524f272922f119fce3bcea6d57fec 100644 (file)
@@ -51,16 +51,19 @@ FILENAME is OUTPUT."
     (with-temp-file (or filename "OUTPUT") (insert text))))
 
 (defun visible-buffer-string ()
     (with-temp-file (or filename "OUTPUT") (insert text))))
 
 (defun visible-buffer-string ()
-  "Same as `buffer-string', but excludes invisible text."
+  "Same as `buffer-string', but excludes invisible text and
+removes any text properties."
   (visible-buffer-substring (point-min) (point-max)))
 
 (defun visible-buffer-substring (start end)
   (visible-buffer-substring (point-min) (point-max)))
 
 (defun visible-buffer-substring (start end)
-  "Same as `buffer-substring', but excludes invisible text."
+  "Same as `buffer-substring-no-properties', but excludes
+invisible text."
   (let (str)
     (while (< start end)
       (let ((next-pos (next-char-property-change start end)))
        (when (not (invisible-p start))
   (let (str)
     (while (< start end)
       (let ((next-pos (next-char-property-change start end)))
        (when (not (invisible-p start))
-         (setq str (concat str (buffer-substring start next-pos))))
+         (setq str (concat str (buffer-substring-no-properties
+                                start next-pos))))
        (setq start next-pos)))
     str))
 
        (setq start next-pos)))
     str))