style.py revision 8237:b83e07b4541d
14202Sbinkertn@umich.edu#! /usr/bin/env python
24202Sbinkertn@umich.edu# Copyright (c) 2006 The Regents of The University of Michigan
34202Sbinkertn@umich.edu# Copyright (c) 2007,2011 The Hewlett-Packard Development Company
44202Sbinkertn@umich.edu# All rights reserved.
54202Sbinkertn@umich.edu#
64202Sbinkertn@umich.edu# Redistribution and use in source and binary forms, with or without
74202Sbinkertn@umich.edu# modification, are permitted provided that the following conditions are
84202Sbinkertn@umich.edu# met: redistributions of source code must retain the above copyright
94202Sbinkertn@umich.edu# notice, this list of conditions and the following disclaimer;
104202Sbinkertn@umich.edu# redistributions in binary form must reproduce the above copyright
114202Sbinkertn@umich.edu# notice, this list of conditions and the following disclaimer in the
124202Sbinkertn@umich.edu# documentation and/or other materials provided with the distribution;
134202Sbinkertn@umich.edu# neither the name of the copyright holders nor the names of its
144202Sbinkertn@umich.edu# contributors may be used to endorse or promote products derived from
154202Sbinkertn@umich.edu# this software without specific prior written permission.
164202Sbinkertn@umich.edu#
174202Sbinkertn@umich.edu# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
184202Sbinkertn@umich.edu# "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
194202Sbinkertn@umich.edu# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
204202Sbinkertn@umich.edu# A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
214202Sbinkertn@umich.edu# OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
224202Sbinkertn@umich.edu# SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
234202Sbinkertn@umich.edu# LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
244202Sbinkertn@umich.edu# DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
254202Sbinkertn@umich.edu# THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
264202Sbinkertn@umich.edu# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
274202Sbinkertn@umich.edu# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
284202Sbinkertn@umich.edu#
294202Sbinkertn@umich.edu# Authors: Nathan Binkert
304202Sbinkertn@umich.edu
314202Sbinkertn@umich.eduimport heapq
324202Sbinkertn@umich.eduimport os
3311053Sandreas.hansson@arm.comimport re
344486Sbinkertn@umich.eduimport sys
355337Sstever@gmail.com
364202Sbinkertn@umich.edufrom os.path import dirname, join as joinpath
3713223Sodanrc@yahoo.com.brfrom itertools import count
385337Sstever@gmail.comfrom mercurial import bdiff, mdiff
395337Sstever@gmail.com
4012726Snikos.nikoleris@arm.comcurrent_dir = dirname(__file__)
4111375Sandreas.hansson@arm.comsys.path.insert(0, current_dir)
4211375Sandreas.hansson@arm.comsys.path.insert(1, joinpath(dirname(current_dir), 'src', 'python'))
435192Ssaidi@eecs.umich.edu
448335Snate@binkert.orgfrom m5.util import neg_inf, pos_inf, Region, Regions
4513939Sodanrc@yahoo.com.brimport sort_includes
468335Snate@binkert.orgfrom file_types import lang_type
478335Snate@binkert.org
489663Suri.wiener@arm.comall_regions = Region(neg_inf, pos_inf)
4911288Ssteve.reinhardt@amd.com
508335Snate@binkert.orgtabsize = 8
5111288Ssteve.reinhardt@amd.comlead = re.compile(r'^([ \t]+)')
5211288Ssteve.reinhardt@amd.comtrail = re.compile(r'([ \t]+)$')
5311288Ssteve.reinhardt@amd.comany_control = re.compile(r'\b(if|while|for)[ \t]*[(]')
5411288Ssteve.reinhardt@amd.comgood_control = re.compile(r'\b(if|while|for) [(]')
5513939Sodanrc@yahoo.com.br
5613939Sodanrc@yahoo.com.brformat_types = set(('C', 'C++'))
5711288Ssteve.reinhardt@amd.com
58def modified_regions(old_data, new_data):
59    regions = Regions()
60    beg = None
61    for pbeg, pend, fbeg, fend in bdiff.blocks(old_data, new_data):
62        if beg is not None and beg != fbeg:
63            regions.append(beg, fbeg)
64        beg = fend
65    return regions
66
67def modregions(wctx, fname):
68    fctx = wctx.filectx(fname)
69    pctx = fctx.parents()
70
71    file_data = fctx.data()
72    lines = mdiff.splitnewlines(file_data)
73    if len(pctx) in (1, 2):
74        mod_regions = modified_regions(pctx[0].data(), file_data)
75        if len(pctx) == 2:
76            m2 = modified_regions(pctx[1].data(), file_data)
77            # only the lines that are new in both
78            mod_regions &= m2
79    else:
80        mod_regions = Regions()
81        mod_regions.add(0, len(lines))
82
83    return mod_regions
84
85class UserInterface(object):
86    def __init__(self, verbose=False, auto=False):
87        self.auto = auto
88        self.verbose = verbose
89
90    def prompt(self, prompt, results, default):
91        if self.auto:
92            return self.auto
93
94        while True:
95            result = self.do_prompt(prompt, results, default)
96            if result in results:
97                return result
98
99class MercurialUI(UserInterface):
100    def __init__(self, ui, *args, **kwargs):
101        super(MercurialUI, self).__init__(*args, **kwargs)
102        self.ui = ui
103
104    def do_prompt(self, prompt, results, default):
105        return self.ui.prompt(prompt, default=default)
106
107    def write(self, string):
108        self.ui.write(string)
109
110class StdioUI(UserInterface):
111    def do_prompt(self, prompt, results, default):
112        return raw_input(prompt) or default
113
114    def write(self, string):
115        sys.stdout.write(string)
116
117class Region(object):
118    def __init__(self, asdf):
119        self.regions = Foo
120
121class Verifier(object):
122    def __init__(self, ui, repo=None):
123        self.ui = ui
124        self.repo = repo
125        if repo is None:
126            self.wctx = None
127
128    def __getattr__(self, attr):
129        if attr in ('prompt', 'write'):
130            return getattr(self.ui, attr)
131
132        if attr == 'wctx':
133            try:
134                wctx = repo.workingctx()
135            except:
136                from mercurial import context
137                wctx = context.workingctx(repo)
138            self.wctx = wctx
139            return wctx
140
141        raise AttributeError
142
143    def open(self, filename, mode):
144        if self.repo:
145            filename = self.repo.wjoin(filename)
146
147        try:
148            f = file(filename, mode)
149        except OSError, msg:
150            print 'could not open file %s: %s' % (filename, msg)
151            return None
152
153        return f
154
155    def skip(self, filename):
156        return lang_type(filename) not in self.languages
157
158    def check(self, filename, regions=all_regions):
159        f = self.open(filename, 'r')
160
161        errors = 0
162        for num,line in enumerate(f):
163            if num not in regions:
164                continue
165            if not self.check_line(line):
166                self.write("invalid %s in %s:%d\n" % \
167                               (self.test_name, filename, num + 1))
168                if self.ui.verbose:
169                    self.write(">>%s<<\n" % line[-1])
170                errors += 1
171        return errors
172
173    def fix(self, filename, regions=all_regions):
174        f = self.open(filename, 'r+')
175
176        lines = list(f)
177
178        f.seek(0)
179        f.truncate()
180
181        for i,line in enumerate(lines):
182            if i in regions:
183                line = self.fix_line(line)
184
185            f.write(line)
186        f.close()
187
188    def apply(self, filename, prompt, regions=all_regions):
189        if not self.skip(filename):
190            errors = self.check(filename, regions)
191            if errors:
192                if prompt(filename, self.fix, regions):
193                    return True
194        return False
195
196
197class Whitespace(Verifier):
198    languages = set(('C', 'C++', 'swig', 'python', 'asm', 'isa', 'scons'))
199    test_name = 'whitespace'
200    def check_line(self, line):
201        match = lead.search(line)
202        if match and match.group(1).find('\t') != -1:
203            return False
204
205        match = trail.search(line)
206        if match:
207            return False
208
209        return True
210
211    def fix_line(self, line):
212        if lead.search(line):
213            newline = ''
214            for i,c in enumerate(line):
215                if c == ' ':
216                    newline += ' '
217                elif c == '\t':
218                    newline += ' ' * (tabsize - len(newline) % tabsize)
219                else:
220                    newline += line[i:]
221                    break
222
223            line = newline
224
225        return line.rstrip() + '\n'
226
227class SortedIncludes(Verifier):
228    languages = sort_includes.default_languages
229    def __init__(self, *args, **kwargs):
230        super(SortedIncludes, self).__init__(*args, **kwargs)
231        self.sort_includes = sort_includes.SortIncludes()
232
233    def check(self, filename, regions=all_regions):
234        f = self.open(filename, 'r')
235
236        lines = [ l.rstrip('\n') for l in f.xreadlines() ]
237        old = ''.join(line + '\n' for line in lines)
238        f.close()
239
240        language = lang_type(filename, lines[0])
241        sort_lines = list(self.sort_includes(lines, filename, language))
242        new = ''.join(line + '\n' for line in sort_lines)
243
244        mod = modified_regions(old, new)
245        modified = mod & regions
246
247        if modified:
248            self.write("invalid sorting of includes\n")
249            if self.ui.verbose:
250                for start, end in modified.regions:
251                    self.write("bad region [%d, %d)\n" % (start, end))
252            return 1
253
254        return 0
255
256    def fix(self, filename, regions=all_regions):
257        f = self.open(filename, 'r+')
258
259        old = f.readlines()
260        lines = [ l.rstrip('\n') for l in old ]
261        language = lang_type(filename, lines[0])
262        sort_lines = list(self.sort_includes(lines, filename, language))
263        new = ''.join(line + '\n' for line in sort_lines)
264
265        f.seek(0)
266        f.truncate()
267
268        for i,line in enumerate(sort_lines):
269            f.write(line)
270            f.write('\n')
271        f.close()
272
273def linelen(line):
274    tabs = line.count('\t')
275    if not tabs:
276        return len(line)
277
278    count = 0
279    for c in line:
280        if c == '\t':
281            count += tabsize - count % tabsize
282        else:
283            count += 1
284
285    return count
286
287class ValidationStats(object):
288    def __init__(self):
289        self.toolong = 0
290        self.toolong80 = 0
291        self.leadtabs = 0
292        self.trailwhite = 0
293        self.badcontrol = 0
294        self.cret = 0
295
296    def dump(self):
297        print '''\
298%d violations of lines over 79 chars. %d of which are 80 chars exactly.
299%d cases of whitespace at the end of a line.
300%d cases of tabs to indent.
301%d bad parens after if/while/for.
302%d carriage returns found.
303''' % (self.toolong, self.toolong80, self.trailwhite, self.leadtabs,
304       self.badcontrol, self.cret)
305
306    def __nonzero__(self):
307        return self.toolong or self.toolong80 or self.leadtabs or \
308               self.trailwhite or self.badcontrol or self.cret
309
310def validate(filename, stats, verbose, exit_code):
311    if lang_type(filename) not in format_types:
312        return
313
314    def msg(lineno, line, message):
315        print '%s:%d>' % (filename, lineno + 1), message
316        if verbose > 2:
317            print line
318
319    def bad():
320        if exit_code is not None:
321            sys.exit(exit_code)
322
323    try:
324        f = file(filename, 'r')
325    except OSError:
326        if verbose > 0:
327            print 'could not open file %s' % filename
328        bad()
329        return
330
331    for i,line in enumerate(f):
332        line = line.rstrip('\n')
333
334        # no carriage returns
335        if line.find('\r') != -1:
336            self.cret += 1
337            if verbose > 1:
338                msg(i, line, 'carriage return found')
339            bad()
340
341        # lines max out at 79 chars
342        llen = linelen(line)
343        if llen > 79:
344            stats.toolong += 1
345            if llen == 80:
346                stats.toolong80 += 1
347            if verbose > 1:
348                msg(i, line, 'line too long (%d chars)' % llen)
349            bad()
350
351        # no tabs used to indent
352        match = lead.search(line)
353        if match and match.group(1).find('\t') != -1:
354            stats.leadtabs += 1
355            if verbose > 1:
356                msg(i, line, 'using tabs to indent')
357            bad()
358
359        # no trailing whitespace
360        if trail.search(line):
361            stats.trailwhite +=1
362            if verbose > 1:
363                msg(i, line, 'trailing whitespace')
364            bad()
365
366        # for c++, exactly one space betwen if/while/for and (
367        if cpp:
368            match = any_control.search(line)
369            if match and not good_control.search(line):
370                stats.badcontrol += 1
371                if verbose > 1:
372                    msg(i, line, 'improper spacing after %s' % match.group(1))
373                bad()
374
375def do_check_style(hgui, repo, *files, **args):
376    """check files for proper m5 style guidelines"""
377    from mercurial import mdiff, util
378
379    auto = args.get('auto', False)
380    if auto:
381        auto = 'f'
382    ui = MercurialUI(hgui, hgui.verbose, auto)
383
384    if files:
385        files = frozenset(files)
386
387    def skip(name):
388        return files and name in files
389
390    def prompt(name, func, regions=all_regions):
391        result = ui.prompt("(a)bort, (i)gnore, or (f)ix?", 'aif', 'a')
392        if result == 'a':
393            return True
394        elif result == 'f':
395            func(repo.wjoin(name), regions)
396
397        return False
398
399    modified, added, removed, deleted, unknown, ignore, clean = repo.status()
400
401    whitespace = Whitespace(ui)
402    sorted_includes = SortedIncludes(ui)
403    for fname in added:
404        if skip(fname):
405            continue
406
407        if whitespace.apply(fname, prompt):
408            return True
409
410        if sorted_includes.apply(fname, prompt):
411            return True
412
413    try:
414        wctx = repo.workingctx()
415    except:
416        from mercurial import context
417        wctx = context.workingctx(repo)
418
419    for fname in modified:
420        if skip(fname):
421            continue
422
423        regions = modregions(wctx, fname)
424
425        if whitespace.apply(fname, prompt, regions):
426            return True
427
428        if sorted_includes.apply(fname, prompt, regions):
429            return True
430
431    return False
432
433def do_check_format(hgui, repo, **args):
434    ui = MercurialUI(hgui, hgui.verbose, auto)
435
436    modified, added, removed, deleted, unknown, ignore, clean = repo.status()
437
438    verbose = 0
439    stats = ValidationStats()
440    for f in modified + added:
441        validate(f, stats, verbose, None)
442
443    if stats:
444        stats.dump()
445        result = ui.prompt("invalid formatting\n(i)gnore or (a)bort?",
446                           'ai', 'a')
447        if result == 'a':
448            return True
449
450    return False
451
452def check_hook(hooktype):
453    if hooktype not in ('pretxncommit', 'pre-qrefresh'):
454        raise AttributeError, \
455              "This hook is not meant for %s" % hooktype
456
457def check_style(ui, repo, hooktype, **kwargs):
458    check_hook(hooktype)
459    args = {}
460
461    try:
462        return do_check_style(ui, repo, **args)
463    except Exception, e:
464        import traceback
465        traceback.print_exc()
466        return True
467
468def check_format(ui, repo, hooktype, **kwargs):
469    check_hook(hooktype)
470    args = {}
471
472    try:
473        return do_check_format(ui, repo, **args)
474    except Exception, e:
475        import traceback
476        traceback.print_exc()
477        return True
478
479try:
480    from mercurial.i18n import _
481except ImportError:
482    def _(arg):
483        return arg
484
485cmdtable = {
486    '^m5style' :
487    ( do_check_style,
488      [ ('a', 'auto', False, _("automatically fix whitespace")) ],
489      _('hg m5style [-a] [FILE]...')),
490    '^m5format' :
491    ( do_check_format,
492      [ ],
493      _('hg m5format [FILE]...')),
494}
495
496if __name__ == '__main__':
497    import getopt
498
499    progname = sys.argv[0]
500    if len(sys.argv) < 2:
501        sys.exit('usage: %s <command> [<command args>]' % progname)
502
503    fixwhite_usage = '%s fixwhite [-t <tabsize> ] <path> [...] \n' % progname
504    chkformat_usage = '%s chkformat <path> [...] \n' % progname
505    chkwhite_usage = '%s chkwhite <path> [...] \n' % progname
506
507    command = sys.argv[1]
508    if command == 'fixwhite':
509        flags = 't:'
510        usage = fixwhite_usage
511    elif command == 'chkwhite':
512        flags = 'nv'
513        usage = chkwhite_usage
514    elif command == 'chkformat':
515        flags = 'nv'
516        usage = chkformat_usage
517    else:
518        sys.exit(fixwhite_usage + chkwhite_usage + chkformat_usage)
519
520    opts, args = getopt.getopt(sys.argv[2:], flags)
521
522    code = 1
523    verbose = 1
524    for opt,arg in opts:
525        if opt == '-n':
526            code = None
527        if opt == '-t':
528            tabsize = int(arg)
529        if opt == '-v':
530            verbose += 1
531
532    if command == 'fixwhite':
533        for filename in args:
534            fixwhite(filename, tabsize)
535    elif command == 'chkwhite':
536        for filename in args:
537            for line,num in checkwhite(filename):
538                print 'invalid whitespace: %s:%d' % (filename, num)
539                if verbose:
540                    print '>>%s<<' % line[:-1]
541    elif command == 'chkformat':
542        stats = ValidationStats()
543        for filename in args:
544            validate(filename, stats=stats, verbose=verbose, exit_code=code)
545
546        if verbose > 0:
547            stats.dump()
548    else:
549        sys.exit("command '%s' not found" % command)
550