gitview: Fix the blame interface.
PATCH 1/2 gitview: Fix the blame interface.
The async reading from the pipe was skipping some of the input lines. Fix the same by making sure that we add the partial content of the previous read to the newly read data.
Signed-off-by Aneesh Kumar K.V <[email protected]>
--- contrib/gitview/gitview | 19 ++++++++++++++++++- 1 files changed, 18 insertions(+), 1 deletions(-)
contrib/gitview/gitview +18 -1
@@ -352,6 +352,7 @@ class AnnotateWindow(object): self.window = gtk.Window(gtk.WINDOW_TOPLEVEL) self.window.set_border_width(0) self.window.set_title("Git repository browser annotation window") + self.prev_read = "" # Use two thirds of the screen by default screen = self.window.get_screen()
@@ -401,7 +402,10 @@ class AnnotateWindow(object): def data_ready(self, source, condition): while (1): try : - buffer = source.read(8192) + # A simple readline doesn't work + # a readline bug ?? + buffer = source.read(100) + except: # resource temporary not available return True
@@ -411,6 +415,19 @@ class AnnotateWindow(object): source.close() return False + if (self.prev_read != ""): + buffer = self.prev_read + buffer + self.prev_read = "" + + if (buffer[len(buffer) -1] != '\n'): + try: + newline_index = buffer.rindex("\n") + except ValueError: + newline_index = 0 + + self.prev_read = buffer[newline_index:(len(buffer))] + buffer = buffer[0:newline_index] + for buff in buffer.split("\n"): annotate_line = re.compile('^([0-9a-f]{40}) (.+) (.+) (.+)$') m = annotate_line.match(buff)
Signature · 1 line
1.5.2.1.255.gca6c0-dirty
PATCH 2/2 gitview: run blame with -C -C
From: Aneesh Kumar K.V <[email protected]>
pass -C -C option to git-blame so that blame browsing works when the data is copied over from other files.
Signed-off-by Aneesh Kumar K.V <[email protected]>
--- contrib/gitview/gitview | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
contrib/gitview/gitview +1 -1
@@ -533,7 +533,7 @@ class AnnotateWindow(object): self.add_file_data(filename, commit_sha1, line_num) - fp = os.popen("git blame --incremental -- " + filename + " " + commit_sha1) + fp = os.popen("git blame --incremental -C -C -- " + filename + " " + commit_sha1) flags = fcntl.fcntl(fp.fileno(), fcntl.F_GETFL) fcntl.fcntl(fp.fileno(), fcntl.F_SETFL, flags | os.O_NONBLOCK) self.io_watch_tag = gobject.io_add_watch(fp, gobject.IO_IN, self.data_ready)
Signature · 1 line
1.5.2.1.255.gca6c0-dirty