bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#2872: marked as done (diff from log-view not working for bzr)


From: Emacs bug Tracking System
Subject: bug#2872: marked as done (diff from log-view not working for bzr)
Date: Tue, 07 Apr 2009 01:55:04 +0000

Your message dated Mon, 06 Apr 2009 21:51:19 -0400
with message-id <address@hidden>
and subject line Re: diff from log-view not working for bzr
has caused the Emacs bug report #2872,
regarding diff from log-view not working for bzr
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact address@hidden
immediately.)


-- 
2872: http://emacsbugs.donarmstrong.com/cgi-bin/bugreport.cgi?bug=2872
Emacs Bug Tracking System
Contact address@hidden with problems
--- Begin Message --- Subject: diff from log-view not working for bzr Date: Thu, 2 Apr 2009 19:09:56 -0700 (PDT)
Do a C-x v l on any file under bzr.

Put the point on some random log entry and type 
d
And empty *vc-diff* window will appear.

The problem is that vc-bzr-diff gets called with the arguments REV1 == REV2.
And that happens because of this code in log-view-diff:

        (log-view-msg-next)
        (setq to (log-view-current-tag))  [1]

Given a log entry like this:

------------------------------------------------------------
revno: 295


`log-view-msg-next' puts the point at the beginning of the ----
line and `log-view-current-tag' returns the version of the _previous_
log entry.  It happens because `log-view-current-tag' first does a 
(forward-line 1) --> point will be at "revno" after that.
And then (re-search-backward log-view-message-re nil t)
which will find the current log entry.

So in conclusion code [1] fails to advance and find the next log
entry.






--- End Message ---
--- Begin Message --- Subject: Re: diff from log-view not working for bzr Date: Mon, 06 Apr 2009 21:51:19 -0400
> Do a C-x v l on any file under bzr.
>
> Put the point on some random log entry and type 
> d
> And empty *vc-diff* window will appear.

I've checked in a fix.


--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]