comparison hgext/churn.py @ 3041:45942bb49194

[churn] Cleanup suggestions from tonfa

Use demandload
Be smarter about the final list creation

3040:f74077473b36

# Aliases map file format is simple one alias per line in the following
# format:
#
# <alias email> <actual email>

import time, sys, signal, os
from mercurial import hg, mdiff, fancyopts, commands, ui, util, templater

def __gather(ui, repo, node1, node2):
    def dirtywork(f, mmap1, mmap2):
        lines = 0

            return

        amap = get_aliases(f)
        f.close()
    
    os.chdir(repo.root)
    stats = gather_stats(ui, repo, amap)

    # make a list of tuples (name, lines) and sort it in descending order
    ordered = stats.items()
    ordered.sort(cmp=lambda x,y:cmp(x[1], y[1]))
    ordered.reverse()

    maximum = ordered[0][1]

    ui.note("Assuming 80 character terminal\n")
    width = 80 - 1

3041:45942bb49194

# Aliases map file format is simple one alias per line in the following
# format:
#
# <alias email> <actual email>

from mercurial.demandload import *
demandload(globals(), 'time sys signal os')
demandload(globals(), 'mercurial:hg,mdiff,fancyopts,commands,ui,util,templater')

def __gather(ui, repo, node1, node2):
    def dirtywork(f, mmap1, mmap2):
        lines = 0

            return

        amap = get_aliases(f)
        f.close()
    
    stats = gather_stats(ui, repo, amap)

    # make a list of tuples (name, lines) and sort it in descending order
    ordered = stats.items()
    ordered.sort(cmp=lambda x,y:cmp(y[1], x[1]))

    maximum = ordered[0][1]

    ui.note("Assuming 80 character terminal\n")
    width = 80 - 1