Hi Rémi,

At Fri, 14 May 2010 14:09:14 +0200,
Rémi Vanicat wrote:

> > +(defun magit-log-all (&optional arg)
> > +  (interactive "P")
> > +  (magit-display-log arg "--all"))
> 
> Do we need to ask for a range, when we are looking for all refs ?
> I believe not, and would wrote:
> (defun magit-log-all ()
>   (magit-display-log () "--all"))

We don't /need/ to but I think it could still be handy, right?

> > +(defun magit-log (&optional arg)
> > +  (interactive "P")
> > +  (magit-display-log arg))
> > +
> >  (defun magit-log-grep (str)
> >    "Search for regexp specified by STR in the commit log."
> >    (interactive "sGrep in commit log: ")
> 
> Does this fix the problem with magit-log listing all the possible refs
> before displaying the logs? May be we need both patch (may be with mine
> changed...) for it to work.

Yea, no extra refs.

Cheers,
Phil

Reply via email to