diff --git a/docs/gitinspector.1 b/docs/gitinspector.1 index e268c5e..d7a752a 100644 --- a/docs/gitinspector.1 +++ b/docs/gitinspector.1 @@ -68,7 +68,7 @@ Show statistics and information in a way that is formatted for grading of studen .PP \fB\-H, \-\-hard\fR[=BOOL] .RS 4 -Track rows and look for duplicates harder; this can be quite slow with big repositories +Track lines and look for duplicates harder; this can be quite slow with big repositories .RE .PP \fB\-l, \-\-list\-file\-types\fR[=BOOL] diff --git a/docs/gitinspector.html b/docs/gitinspector.html index dbfa239..ce5b909 100644 --- a/docs/gitinspector.html +++ b/docs/gitinspector.html @@ -14,7 +14,7 @@
-H, --hard[=BOOL]
- Track rows and look for duplicates harder; this can be quite slow with big repositories + Track lines and look for duplicates harder; this can be quite slow with big repositories
-l, --list-file-types[=BOOL]
diff --git a/docs/gitinspector.txt b/docs/gitinspector.txt index 9134805..31e5373 100644 --- a/docs/gitinspector.txt +++ b/docs/gitinspector.txt @@ -41,7 +41,7 @@ Mandatory arguments to long options are mandatory for short options too. Boolean Show statistics and information in a way that is formatted for grading of student projects; this is the same as supplying the options *-HlmrTw* *-H, --hard*[=BOOL]:: - Track rows and look for duplicates harder; this can be quite slow with big repositories + Track lines and look for duplicates harder; this can be quite slow with big repositories *-l, --list-file-types*[=BOOL]:: List all the file extensions available in the current branch of the repository diff --git a/gitinspector/blame.py b/gitinspector/blame.py index 317d3f9..0b48364 100644 --- a/gitinspector/blame.py +++ b/gitinspector/blame.py @@ -31,7 +31,7 @@ NUM_THREADS = multiprocessing.cpu_count() class BlameEntry(object): - rows = 0 + lines = 0 skew = 0 # Used when calculating average code age. comments = 0 @@ -82,7 +82,7 @@ def __handle_blamechunk_content__(self, content): self.blames[(author, self.filename)] = BlameEntry() self.blames[(author, self.filename)].comments += comments - self.blames[(author, self.filename)].rows += 1 + self.blames[(author, self.filename)].lines += 1 if (self.blamechunk_time - self.changes.first_commit_date).days > 0: self.blames[(author, self.filename)].skew += ((self.changes.last_commit_date - self.blamechunk_time).days / @@ -92,18 +92,18 @@ def __handle_blamechunk_content__(self, content): def run(self): git_blame_r = subprocess.Popen(self.blame_command, bufsize=1, stdout=subprocess.PIPE).stdout - rows = git_blame_r.readlines() + lines = git_blame_r.readlines() git_blame_r.close() self.__clear_blamechunk_info__() #pylint: disable=W0201 - for j in range(0, len(rows)): - row = rows[j].decode("utf-8", "replace").strip() - keyval = row.split(" ", 2) + for j in range(0, len(lines)): + line = lines[j].decode("utf-8", "replace").strip() + keyval = line.split(" ", 2) if self.blamechunk_is_last: - self.__handle_blamechunk_content__(row) + self.__handle_blamechunk_content__(line) self.__clear_blamechunk_info__() elif keyval[0] == "boundary": self.blamechunk_is_prior = True @@ -118,7 +118,7 @@ def run(self): __thread_lock__.release() # Lock controlling the number of threads running -PROGRESS_TEXT = N_("Checking how many rows belong to each author (2 of 2): {0:.0f}%") +PROGRESS_TEXT = N_("Checking how many lines belong to each author (2 of 2): {0:.0f}%") class Blame(object): def __init__(self, repo, hard, useweeks, changes): @@ -134,18 +134,18 @@ def __init__(self, repo, hard, useweeks, changes): if repo != None: progress_text = "[%s] " % repo.name + progress_text - for i, row in enumerate(lines): - row = row.strip().decode("unicode_escape", "ignore") - row = row.encode("latin-1", "replace") - row = row.decode("utf-8", "replace").strip("\"").strip("'").strip() + for i, line in enumerate(lines): + line = line.strip().decode("unicode_escape", "ignore") + line = line.encode("latin-1", "replace") + line = line.decode("utf-8", "replace").strip("\"").strip("'").strip() - if FileDiff.get_extension(row) in extensions.get_located() and \ - FileDiff.is_valid_extension(row) and not filtering.set_filtered(FileDiff.get_filename(row)): + if FileDiff.get_extension(line) in extensions.get_located() and \ + FileDiff.is_valid_extension(line) and not filtering.set_filtered(FileDiff.get_filename(line)): blame_command = filter(None, ["git", "blame", "--line-porcelain", "-w"] + \ (["-C", "-C", "-M"] if hard else []) + - [interval.get_since(), interval.get_ref(), "--", row]) - thread = BlameThread(useweeks, changes, blame_command, FileDiff.get_extension(row), - self.blames, row.strip()) + [interval.get_since(), interval.get_ref(), "--", line]) + thread = BlameThread(useweeks, changes, blame_command, FileDiff.get_extension(line), + self.blames, line.strip()) thread.daemon = True thread.start() @@ -177,10 +177,10 @@ def is_revision(string): return revision.group(1).strip() @staticmethod - def get_stability(author, blamed_rows, changes): + def get_stability(author, blamed_lines, changes): if author in changes.get_authorinfo_list(): author_insertions = changes.get_authorinfo_list()[author].insertions - return 100 if author_insertions == 0 else 100.0 * blamed_rows / author_insertions + return 100 if author_insertions == 0 else 100.0 * blamed_lines / author_insertions return 100 @staticmethod @@ -194,7 +194,7 @@ def get_summed_blames(self): if summed_blames.get(i[0][0], None) == None: summed_blames[i[0][0]] = BlameEntry() - summed_blames[i[0][0]].rows += i[1].rows + summed_blames[i[0][0]].lines += i[1].lines summed_blames[i[0][0]].skew += i[1].skew summed_blames[i[0][0]].comments += i[1].comments diff --git a/gitinspector/format.py b/gitinspector/format.py index 505da03..05ee1c6 100644 --- a/gitinspector/format.py +++ b/gitinspector/format.py @@ -106,8 +106,8 @@ def output_header(repos): repos_string, localization.get_date()), show_minor_authors=_("Show minor authors"), hide_minor_authors=_("Hide minor authors"), - show_minor_rows=_("Show rows with minor work"), - hide_minor_rows=_("Hide rows with minor work"))) + show_minor_lines=_("Show lines with minor work"), + hide_minor_lines=_("Hide lines with minor work"))) elif __selected_format__ == "json": print("{\n\t\"gitinspector\": {") print("\t\t\"version\": \"" + version.__version__ + "\",") diff --git a/gitinspector/gitinspector.py b/gitinspector/gitinspector.py index 7149294..279e4ca 100644 --- a/gitinspector/gitinspector.py +++ b/gitinspector/gitinspector.py @@ -75,7 +75,7 @@ def process(self, repos): summed_metrics += MetricsLogic() if sys.stdout.isatty() and format.is_interactive_format(): - terminal.clear_row() + terminal.clear_line() else: os.chdir(previous_directory) diff --git a/gitinspector/help.py b/gitinspector/help.py index 447dcb5..6721bc0 100644 --- a/gitinspector/help.py +++ b/gitinspector/help.py @@ -45,7 +45,7 @@ is formatted for grading of student projects; this is the same as supplying the options -HlmrTw - -H, --hard[=BOOL] track rows and look for duplicates harder; + -H, --hard[=BOOL] track lines and look for duplicates harder; this can be quite slow with big repositories -l, --list-file-types[=BOOL] list all the file extensions available in the current branch of the repository diff --git a/gitinspector/html/html.header b/gitinspector/html/html.header index 8cba24f..1c7672d 100644 --- a/gitinspector/html/html.header +++ b/gitinspector/html/html.header @@ -10,19 +10,19 @@