Patchwork [1,of,2,v2] crecord: delete commented line

login
register
mail settings
Submitter Nathan Goldbaum
Date Sept. 15, 2016, 6:49 p.m.
Message ID <dec5dfcbd772a9f51674.1473965358@rous2.ncsa.illinois.edu>
Download mbox | patch
Permalink /patch/16638/
State Accepted
Headers show

Comments

Nathan Goldbaum - Sept. 15, 2016, 6:49 p.m.
# HG changeset patch
# User Nathan Goldbaum <ngoldbau@illinois.edu>
# Date 1473871187 18000
#      Wed Sep 14 11:39:47 2016 -0500
# Node ID dec5dfcbd772a9f51674067ad18dff6d9bad78cf
# Parent  2c302c6544511b37abe61ee2cc5278e4f61d5e71
crecord: delete commented line
Pierre-Yves David - Sept. 16, 2016, 10:03 a.m.
On 09/15/2016 08:49 PM, Nathan Goldbaum wrote:
> # HG changeset patch
> # User Nathan Goldbaum <ngoldbau@illinois.edu>
> # Date 1473871187 18000
> #      Wed Sep 14 11:39:47 2016 -0500
> # Node ID dec5dfcbd772a9f51674067ad18dff6d9bad78cf
> # Parent  2c302c6544511b37abe61ee2cc5278e4f61d5e71
> crecord: delete commented line

I've pushed that one as obviously correct. I think I'm okay with patch 
2, but'll let other people time to express opinion on it.

Patch

diff -r 2c302c654451 -r dec5dfcbd772 mercurial/crecord.py
--- a/mercurial/crecord.py	Wed Sep 14 17:12:39 2016 +0200
+++ b/mercurial/crecord.py	Wed Sep 14 11:39:47 2016 -0500
@@ -719,7 +719,7 @@  class curseschunkselector(object):
         "scroll the screen to fully show the currently-selected"
         selstart = self.selecteditemstartline
         selend = self.selecteditemendline
-        #selnumlines = selend - selstart
+
         padstart = self.firstlineofpadtoprint
         padend = padstart + self.yscreensize - self.numstatuslines - 1
         # 'buffered' pad start/end values which scroll with a certain