<div class="gmail_quote">2009/5/13 Ian Clatworthy <span dir="ltr">&lt;<a href="mailto:ian.clatworthy@canonical.com">ian.clatworthy@canonical.com</a>&gt;</span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<font color="#888888">Ian Clatworthy has voted comment.<br>
Status is now: Semi-approved<br>
Comment:<br>
The behaviour is documented and by design: it starts at the end revision and walks the graph until it finds the start revision. </font></blockquote><div><br>I apologise for my ignorance, I vaguely remember a discussion about this when you made the changes.<br>
 </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><font color="#888888">Please re-read the help for log. </font></blockquote><div> </div>
<div>thanks for the pointer, its been a while since I read what has now become a book :) nice and complete.<br> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<font color="#888888">If there&#39;s a genuine use case for wanting filtering done in the way you expected, let me know. </font></blockquote><div><br>I was just surprised that generating the graph produces different output than not, and concerned that it may be an oversight. This was my way of clarifying it, and to that end I&#39;ll submit tests for the designed behaviour shortly.<br>
 </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><font color="#888888">Otherwise, please reject this patch.</font></blockquote><div><br>
</div></div>bb:reject<br><br><br><div style="margin-left: 40px;">2009/5/11 John Arbash Meinel <span dir="ltr">&lt;<a href="mailto:john@arbash-meinel.com">john@arbash-meinel.com</a>&gt;</span><br></div><div style="margin-left: 40px;" class="im">
&gt; As you can see it skips some revisions. After doing some debugging,<br>
&gt; I realised that it skips them because they are not ancestors of the<br>
&gt; end-revision.<br><br>
</div><div style="margin-left: 40px;">If they aren&#39;t in the ancestry, why would they be displayed?<br></div>
<br>well to me it originally looked like:<br><br>F<br>|\<br>|E<br>| |<br>|D <br>|/ <br>C<br>|<br>B<br>|<br>A<br><br>because of how log displays it. <br>thus I expected log A..D to show A,B,C,D<br><br>but it is actually:<br>
<br>
F<br>
|\<br>
CE<br>
| |<br>
BD <br>
|/ <br>
A<br>
<br>So it shows A,D<br><br>regards<br>marius<br>(Whew, I normally skip over e-mails with diagrams like above because they require too much thinking, and now I make one. :)<br><br><br><br>