Path: utzoo!utgpu!jarvis.csri.toronto.edu!rutgers!tut.cis.ohio-state.edu!SUN.COM!peck
From: peck@SUN.COM (Jeff Peck)
Newsgroups: gnu.emacs.bug
Subject: Re: prev-complex-command in sun.el
Message-ID: <8909120125.AA09299@denali.sun.com>
Date: 12 Sep 89 01:25:24 GMT
References: <8909111710.AA29891@sn1987a.compass.com>
Sender: daemon@tut.cis.ohio-state.edu
Distribution: gnu
Organization: GNUs Not Usenet
Lines: 24

Actually, the functions in sun.el should mostly get absorbed into
the regular emacs functions.
In this case, i'd prefer to see "repeat-complex-command" handle this
functionality, and then remove completely this function.

(defun prev-complex-command ()
  "Select Previous-complex-command"
  (interactive)
  (if (zerop (minibuffer-depth))
      (repeat-complex-command 1)
    (previous-complex-command 1)))

That is, repeat-complex-command should check to see if it is already being
called, and if so, just call into previous-complex-command.
This way, one keybinding serves for both functions.

Instead of checking minibuffer-depth, repeat-complex-command should probably
dynmamic bind something: 
(defun repeat-complex-command (...)
   (if already-in-repeat-complex-command
	  (previous-complex-command ...)
	(let ((already-in-repeat-complex-command t))
	  ...))
	)