unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* icomplete.el patch to accommodate addition of internal-complete-buffer
@ 2006-09-22 20:07 Ken Manheimer
  2006-09-23 18:41 ` Chong Yidong
  2006-09-23 23:37 ` Ken Manheimer
  0 siblings, 2 replies; 4+ messages in thread
From: Ken Manheimer @ 2006-09-22 20:07 UTC (permalink / raw)


[-- Attachment #1: Type: text/plain, Size: 1056 bytes --]

the attached patch reenables icomplete operation during interactive
buffer selection.  the ChangeLog entry is included below my signature,
and it's also attached along with the patch.

i would like to know about any other specialized completion
tables/functions that would be appropriate - and trivial - to include
among those that icomplete accepts.  i would even more like some way
for icomplete to automatically recognize those functions, but don't
know the territory well enough to be able to say whether that's even a
reasonable possibility.

in any case, would someone check this in?
-- 
ken
ken.manheimer@gmail.com
http://myriadicity.net

2006-09-22  Ken Manheimer  <ken.manheimer@gmail.com>

	* icomplete.el (icomplete-with-completion-tables): List of
	specialized completion tables with which icomplete should
	operate.  Include the new `internal-complete-buffer', so icomplete
	works with interactive buffer selection.
	(icomplete-simple-completing-p): Add acceptance of specialized
	completion tables listed in icomplete-with-completion-tables.

[-- Attachment #2: icomplete-patch.txt --]
[-- Type: text/plain, Size: 1201 bytes --]

--- icomplete.el	17 Feb 2006 16:54:35 -0500	1.38
+++ icomplete.el	22 Sep 2006 15:52:47 -0400	
@@ -157,6 +157,12 @@
 					 (< (length x) (length y))))
 			       ", ")
 		    ">"))))))
+;;;_  = icomplete-with-completion-tables
+(defvar icomplete-with-completion-tables '(internal-complete-buffer)
+  "Specialized completion tables with which icomplete should operate.
+
+Icomplete does not operate with any specialized completion tables
+except those on this list.")
 
 ;;;_ > icomplete-mode (&optional prefix)
 ;;;###autoload
@@ -184,8 +190,9 @@
   (and (window-minibuffer-p (selected-window))
        (not executing-kbd-macro)
        minibuffer-completion-table
-       ;; (or minibuffer-completing-file-name
-       (not (functionp minibuffer-completion-table)))) ;; )
+       (or (not (functionp minibuffer-completion-table))
+           (member minibuffer-completion-table
+                   icomplete-for-completion-tables))))
 
 ;;;_ > icomplete-minibuffer-setup ()
 (defun icomplete-minibuffer-setup ()
@@ -326,6 +333,7 @@
 ;;;_* Local emacs vars.
 ;;;Local variables:
 ;;;allout-layout: (-2 :)
+;;;allout-widgets-mode-inhibit: t
 ;;;End:
 
 ;; arch-tag: 339ec25a-0741-4eb6-be63-997532e89b0f

[-- Attachment #3: icomplete-ChangeLog.txt --]
[-- Type: text/plain, Size: 410 bytes --]

2006-09-22  Ken Manheimer  <ken.manheimer@gmail.com>

	* icomplete.el (icomplete-with-completion-tables): List of
	specialized completion tables with which icomplete should
	operate.  Include the new `internal-complete-buffer', so icomplete
	works with interactive buffer-selection.
	(icomplete-simple-completing-p): Add acceptance of specialized
	completion tables listed in icomplete-with-completion-tables.

[-- Attachment #4: Type: text/plain, Size: 142 bytes --]

_______________________________________________
Emacs-devel mailing list
Emacs-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-devel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: icomplete.el patch to accommodate addition of internal-complete-buffer
  2006-09-22 20:07 icomplete.el patch to accommodate addition of internal-complete-buffer Ken Manheimer
@ 2006-09-23 18:41 ` Chong Yidong
  2006-09-23 23:37 ` Ken Manheimer
  1 sibling, 0 replies; 4+ messages in thread
From: Chong Yidong @ 2006-09-23 18:41 UTC (permalink / raw)
  Cc: Emacs-Devel

"Ken Manheimer" <ken.manheimer@gmail.com> writes:

> the attached patch reenables icomplete operation during interactive
> buffer selection.  the ChangeLog entry is included below my signature,
> and it's also attached along with the patch.
>
> i would like to know about any other specialized completion
> tables/functions that would be appropriate - and trivial - to include
> among those that icomplete accepts.  i would even more like some way
> for icomplete to automatically recognize those functions, but don't
> know the territory well enough to be able to say whether that's even a
> reasonable possibility.
>
> in any case, would someone check this in?

Done.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: icomplete.el patch to accommodate addition of internal-complete-buffer
  2006-09-22 20:07 icomplete.el patch to accommodate addition of internal-complete-buffer Ken Manheimer
  2006-09-23 18:41 ` Chong Yidong
@ 2006-09-23 23:37 ` Ken Manheimer
  2006-09-24 13:49   ` Chong Yidong
  1 sibling, 1 reply; 4+ messages in thread
From: Ken Manheimer @ 2006-09-23 23:37 UTC (permalink / raw)



[-- Attachment #1.1: Type: text/plain, Size: 1630 bytes --]

i made a mistake in yesterdays patch, changing the new variable name but not
the reference to it.  the attached patch rectifies that, and removes a
superfluous local variable setting which would set off the
local-variable-setting-warning for everyone else in the world but me.  the
attached ChangeLog entry details the changes.  sorry about the extra work!

ken

On 9/22/06, Ken Manheimer <ken.manheimer@gmail.com> wrote:
>
> the attached patch reenables icomplete operation during interactive
> buffer selection.  the ChangeLog entry is included below my signature,
> and it's also attached along with the patch.
>
> i would like to know about any other specialized completion
> tables/functions that would be appropriate - and trivial - to include
> among those that icomplete accepts.  i would even more like some way
> for icomplete to automatically recognize those functions, but don't
> know the territory well enough to be able to say whether that's even a
> reasonable possibility.
>
> in any case, would someone check this in?
> --
> ken
> ken.manheimer@gmail.com
> http://myriadicity.net
>
> 2006-09-22  Ken Manheimer  <ken.manheimer@gmail.com>
>
>         * icomplete.el (icomplete-with-completion-tables): List of
>         specialized completion tables with which icomplete should
>         operate.  Include the new `internal-complete-buffer', so icomplete
>         works with interactive buffer selection.
>         (icomplete-simple-completing-p): Add acceptance of specialized
>         completion tables listed in icomplete-with-completion-tables.
>
>
>


-- 
ken
ken.manheimer@gmail.com
http://myriadicity.net

[-- Attachment #1.2: Type: text/html, Size: 2592 bytes --]

[-- Attachment #2: icomplete-patch.txt --]
[-- Type: text/plain, Size: 636 bytes --]

--- icomplete.el	23 Sep 2006 17:27:18 -0400	1.39
+++ icomplete.el	23 Sep 2006 17:34:15 -0400	
@@ -192,7 +192,7 @@
        minibuffer-completion-table
        (or (not (functionp minibuffer-completion-table))
            (member minibuffer-completion-table
-                   icomplete-for-completion-tables))))
+                   icomplete-with-completion-tables))))
 
 ;;;_ > icomplete-minibuffer-setup ()
 (defun icomplete-minibuffer-setup ()
@@ -333,7 +333,6 @@
 ;;;_* Local emacs vars.
 ;;;Local variables:
 ;;;allout-layout: (-2 :)
-;;;allout-widgets-mode-inhibit: t
 ;;;End:
 
 ;; arch-tag: 339ec25a-0741-4eb6-be63-997532e89b0f

[-- Attachment #3: icomplete-ChangeLog.txt --]
[-- Type: text/plain, Size: 236 bytes --]

2006-09-23  Ken Manheimer  <ken.manheimer@gmail.com>

	* icomplete.el (icomplete-simple-completing-p): Use the correct
	name for the new variable, `icomplete-with-completion-tables'.
	(file local variables): Remove superfluous setting.

[-- Attachment #4: Type: text/plain, Size: 142 bytes --]

_______________________________________________
Emacs-devel mailing list
Emacs-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-devel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: icomplete.el patch to accommodate addition of internal-complete-buffer
  2006-09-23 23:37 ` Ken Manheimer
@ 2006-09-24 13:49   ` Chong Yidong
  0 siblings, 0 replies; 4+ messages in thread
From: Chong Yidong @ 2006-09-24 13:49 UTC (permalink / raw)
  Cc: Emacs-Devel

"Ken Manheimer" <ken.manheimer@gmail.com> writes:

> i made a mistake in yesterdays patch, changing the new variable name but not
> the reference to it.  the attached patch rectifies that, and removes a
> superfluous local variable setting which would set off the
> local-variable-setting-warning for everyone else in the world but me.  the
> attached ChangeLog entry details the changes.  sorry about the extra work!

Checked in.  Thanks.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2006-09-24 13:49 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-09-22 20:07 icomplete.el patch to accommodate addition of internal-complete-buffer Ken Manheimer
2006-09-23 18:41 ` Chong Yidong
2006-09-23 23:37 ` Ken Manheimer
2006-09-24 13:49   ` Chong Yidong

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).