GNU bug report logs - #4369
expunge references to lisp-re in re-builder

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Juanma Barranquero <lekktu <at> gmail.com>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#4369: closed (expunge references to lisp-re in re-builder)
Date: Tue, 27 Jul 2010 23:44:02 +0000

[Message part 1 (text/plain, inline)]

Your message dated Wed, 28 Jul 2010 01:43:03 +0200
with message-id <AANLkTimAN+ajX0NaE0OjLfPoOv9qb0rZPkcW9Nu_--xD <at> mail.gmail.com>
and subject line Re: bug#4369: expunge references to lisp-re in re-builder
has caused the GNU bug report #4369,
regarding expunge references to lisp-re in re-builder
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
4369: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=4369
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: "Aaron S. Hawley" <ashawley <at> burlingtontelecom.net>
To: bug-gnu-emacs <at> gnu.org
Subject: expunge references to lisp-re in re-builder
Date: Tue, 8 Sep 2009 00:35:52 -0400

[Message part 3 (text/plain, inline)]

The lisp-re library never made it out of Detlev Zundel's private
collection.  I suggest that all references to it be removed.  That way
people can avoid getting error messages like: "Cannot open load file:
lisp-re".

--- re-builder.el	08 Sep 2009 00:16:11 -0400	1.47
+++ re-builder.el	08 Sep 2009 00:18:41 -0400	
@@ -61,14 +61,12 @@
 ;; this limit allowing an easy way to see all matches.

 ;; Currently `re-builder' understands five different forms of input,
-;; namely `read', `string', `rx', `sregex' and `lisp-re' syntax.  Read
+;; namely `read', `string', `rx', and `sregex' syntax.  Read
 ;; syntax and string syntax are both delimited by `"'s and behave
 ;; according to their name.  With the `string' syntax there's no need
 ;; to escape the backslashes and double quotes simplifying the editing
 ;; somewhat.  The other three allow editing of symbolic regular
-;; expressions supported by the packages of the same name.  (`lisp-re'
-;; is a package by me and its support may go away as it is nearly the
-;; same as the `sregex' package in Emacs)
+;; expressions supported by the packages of the same name.

 ;; Editing symbolic expressions is done through a major mode derived
 ;; from `emacs-lisp-mode' so you'll get all the good stuff like
@@ -128,12 +126,11 @@

 (defcustom reb-re-syntax 'read
   "Syntax for the REs in the RE Builder.
-Can either be `read', `string', `sregex', `lisp-re', `rx'."
+Can either be `read', `string', `sregex', `rx'."
   :group 're-builder
   :type '(choice (const :tag "Read syntax" read)
 		 (const :tag "String syntax" string)
 		 (const :tag "`sregex' syntax" sregex)
-		 (const :tag "`lisp-re' syntax" lisp-re)
 		 (const :tag "`rx' syntax" rx)))

 (defcustom reb-auto-match-limit 200
@@ -281,8 +278,7 @@
 (define-derived-mode reb-lisp-mode
   emacs-lisp-mode "RE Builder Lisp"
   "Major mode for interactively building symbolic Regular Expressions."
-  (cond ((eq reb-re-syntax 'lisp-re)	; Pull in packages
-	 (require 'lisp-re))		; as needed
+  (cond					; Pull in packages as needed
 	((eq reb-re-syntax 'sregex)	; sregex is not autoloaded
 	 (require 'sregex))		; right now..
 	((eq reb-re-syntax 'rx)		; rx-to-string is autoloaded
@@ -329,7 +325,7 @@

 (defsubst reb-lisp-syntax-p ()
   "Return non-nil if RE Builder uses a Lisp syntax."
-  (memq reb-re-syntax '(lisp-re sregex rx)))
+  (memq reb-re-syntax '(sregex rx)))

 (defmacro reb-target-binding (symbol)
   "Return binding for SYMBOL in the RE Builder target buffer."
@@ -489,10 +485,10 @@
    (list (intern
 	  (completing-read "Select syntax: "
 			   (mapcar (lambda (el) (cons (symbol-name el) 1))
-				   '(read string lisp-re sregex rx))
+				   '(read string sregex rx))
 			   nil t (symbol-name reb-re-syntax)))))

-  (if (memq syntax '(read string lisp-re sregex rx))
+  (if (memq syntax '(read string sregex rx))
       (let ((buffer (get-buffer reb-buffer)))
 	(setq reb-re-syntax syntax)
 	(when buffer
@@ -616,10 +612,7 @@

 (defun reb-cook-regexp (re)
   "Return RE after processing it according to `reb-re-syntax'."
-  (cond ((eq reb-re-syntax 'lisp-re)
-	 (when (fboundp 'lre-compile-string)
-	   (lre-compile-string (eval (car (read-from-string re))))))
-	((eq reb-re-syntax 'sregex)
+  (cond ((eq reb-re-syntax 'sregex)
 	 (apply 'sregex (eval (car (read-from-string re)))))
 	((eq reb-re-syntax 'rx)
 	 (rx-to-string (eval (car (read-from-string re)))))


-- 
In general, we reserve the right to have a poor
memory--the computer, however, is supposed to
remember!  Poor computer.  -- Guy Lewis Steele Jr.

[re-builder.el.diff (application/octet-stream, attachment)]

[Message part 5 (message/rfc822, inline)]

From: Juanma Barranquero <lekktu <at> gmail.com>
To: "Aaron S. Hawley" <ashawley <at> burlingtontelecom.net>
Cc: 4369-done <at> debbugs.gnu.org
Subject: Re: bug#4369: expunge references to lisp-re in re-builder
Date: Wed, 28 Jul 2010 01:43:03 +0200

On Tue, Sep 8, 2009 at 06:35, Aaron S. Hawley
<ashawley <at> burlingtontelecom.net> wrote:

> The lisp-re library never made it out of Detlev Zundel's private
> collection.  I suggest that all references to it be removed.  That way
> people can avoid getting error messages like: "Cannot open load file:
> lisp-re".

I've added a ChangeLog entry to your patch and committed it with
minimal changes.

    Juanma

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.