reports/Builder__LintReport.st
author Jan Vrany <jan.vrany@fit.cvut.cz>
Fri, 01 Mar 2013 10:42:50 +0100
changeset 117 d7f87303b984
parent 84 77e12dbe97d3
child 118 7e93ef8c5417
permissions -rw-r--r--
class: Builder::LintReport added: #cmdlineOptionRuleset #setupRulesFrom: changed: #setUp category of:
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
71
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
     1
"{ Package: 'stx:goodies/builder/reports' }"
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
     2
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
     3
"{ NameSpace: Builder }"
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
     4
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
     5
Report subclass:#LintReport
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
     6
	instanceVariableNames:'environment rules'
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
     7
	classVariableNames:''
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
     8
	poolDictionaries:''
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
     9
	category:'Builder-Reports'
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    10
!
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    11
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    12
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    13
!LintReport class methodsFor:'class initialization'!
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    14
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    15
initialize
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    16
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    17
    Smalltalk loadPackage: 'stx:goodies/refactoryBrowser/lint'.
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    18
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    19
    "Modified: / 01-04-2011 / 12:58:47 / Jan Vrany <jan.vrany@fit.cvut.cz>"
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    20
! !
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    21
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    22
!LintReport methodsFor:'accessing - defaults'!
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    23
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    24
defaultFileSuffix
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    25
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    26
    ^ 'Lint'
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    27
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    28
    "Modified: / 08-10-2011 / 10:49:30 / Jan Vrany <jan.vrany@fit.cvut.cz>"
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    29
!
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    30
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    31
defaultFormat
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    32
    "raise an error: must be redefined in concrete subclass(es)"
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    33
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    34
    ^ LintReportFormat::PMD new
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    35
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    36
    "Modified: / 25-11-2011 / 22:06:33 / Jan Vrany <jan.vrany@fit.cvut.cz>"
79
889d3747a0a8 Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 71
diff changeset
    37
!
889d3747a0a8 Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 71
diff changeset
    38
889d3747a0a8 Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 71
diff changeset
    39
defaultName
889d3747a0a8 Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 71
diff changeset
    40
889d3747a0a8 Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 71
diff changeset
    41
    environment isNil ifTrue:[^super defaultName].
889d3747a0a8 Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 71
diff changeset
    42
    ^environment label
889d3747a0a8 Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 71
diff changeset
    43
889d3747a0a8 Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 71
diff changeset
    44
    "Modified: / 25-11-2011 / 22:06:33 / Jan Vrany <jan.vrany@fit.cvut.cz>"
889d3747a0a8 Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 71
diff changeset
    45
    "Created: / 13-01-2012 / 12:43:07 / Jan Vrany <jan.vrany@fit.cvut.cz>"
71
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    46
! !
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    47
117
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
    48
!LintReport methodsFor:'command line options'!
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
    49
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
    50
cmdlineOptionRuleset
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
    51
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
    52
    ^CmdLineOption new
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
    53
        short: $s;
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
    54
        long: 'ruleset';
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
    55
        description: 'Rule set ';
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
    56
        action:[:option |
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
    57
            self setupRulesFrom: option.
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
    58
        ];
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
    59
        yourself
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
    60
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
    61
    "Created: / 28-02-2013 / 23:13:57 / Jan Vrany <jan.vrany@fit.cvut.cz>"
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
    62
! !
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
    63
71
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    64
!LintReport methodsFor:'generating'!
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    65
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    66
generateClass: aClass
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    67
        | sourceStream sourceName |
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    68
        sourceStream := WriteStream on: String new.
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    69
        sourceName := self encodeFilename: (self sourceFilenameFor: aClass).
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    70
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    71
        format writeFile: sourceName with: [
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    72
            self generateClass: aClass source: sourceStream.
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    73
            self generateClass: aClass class source: sourceStream.
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    74
        ].
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    75
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    76
        format writeSource: sourceStream contents to: sourceName
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    77
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    78
    "Created: / 06-10-2011 / 23:54:37 / Jan Vrany <jan.vrany@fit.cvut.cz>"
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    79
!
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    80
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    81
generateClass: aClass selector: aSelector source: sourceStream
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    82
        | offset source matching |
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    83
        offset := self
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    84
                lineAndColumn: sourceStream contents
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    85
                at: sourceStream position.
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    86
        sourceStream
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    87
                nextPutAll: (source := self convert: (aClass sourceCodeAt: aSelector));
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    88
                nextPut: Character lf; nextPut: Character lf.
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    89
        matching := rules select: [ :each | 
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    90
                (self isSelectorEnvironment: each result)
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    91
                        and: [ each result includesSelector: aSelector in: aClass ] ].
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    92
        self generateViolations: matching class: aClass selector: aSelector source: source offset: offset
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    93
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    94
    "Created: / 07-10-2011 / 11:04:12 / Jan Vrany <jan.vrany@fit.cvut.cz>"
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    95
!
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    96
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    97
generateClass: aClass source: sourceStream 
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    98
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
    99
        | offset source matching |
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   100
        offset := self
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   101
                lineAndColumn: sourceStream contents
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   102
                at: sourceStream position.
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   103
        sourceStream
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   104
                nextPutAll: (source := self convert: aClass definition);
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   105
                nextPut: Character lf; nextPut: Character lf.
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   106
        (environment definesClass: aClass) ifTrue: [
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   107
                matching := rules select: [ :rule |
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   108
                        (self isClassEnvironment: rule result)
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   109
                                and: [ rule result includesClass: aClass ] ].
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   110
                self generateViolations: matching class: aClass selector: nil  source: source offset: offset ].
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   111
        (environment selectorsForClass: aClass) asSortedCollection
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   112
                do: [ :selector | self generateClass: aClass selector: selector source: sourceStream]
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   113
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   114
    "Created: / 07-10-2011 / 10:29:42 / Jan Vrany <jan.vrany@fit.cvut.cz>"
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   115
!
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   116
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   117
generateViolations: aCollection class: aClass selector: aSelector source: aString offset: aPoint
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   118
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   119
    aCollection do: [ :rule |
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   120
        | interval start stop |
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   121
    
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   122
        interval := (rule result selectionIntervalFor: aString) ifNil: [ 1 to: aString size ].
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   123
        start := self lineAndColumn: aString at: interval first.
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   124
        stop  := self lineAndColumn: aString at: interval last.
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   125
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   126
        format writeViolation: rule
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   127
                   class: aClass selector: aSelector
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   128
               startLine: aPoint x + start x column: aPoint y + start y - 1
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   129
                stopLine: aPoint x + stop x  column: aPoint y + stop  y - 1.
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   130
    ]
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   131
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   132
    "Created: / 25-11-2011 / 22:23:09 / Jan Vrany <jan.vrany@fit.cvut.cz>"
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   133
! !
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   134
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   135
!LintReport methodsFor:'initialization'!
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   136
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   137
setupForClasses: classes
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   138
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   139
    environment := BrowserEnvironment new forClasses: classes.
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   140
    environment label: name
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   141
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   142
    "Created: / 04-08-2011 / 14:40:31 / Jan Vrany <jan.vrany@fit.cvut.cz>"
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   143
!
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   144
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   145
setupForPackages: packages
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   146
79
889d3747a0a8 Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 71
diff changeset
   147
    packages isEmpty ifTrue:[^self].
71
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   148
    environment := PackageEnvironment 
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   149
                    onEnvironment: BrowserEnvironment new
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   150
                    packageNames: packages.
79
889d3747a0a8 Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 71
diff changeset
   151
    name isNil ifTrue:[
889d3747a0a8 Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 71
diff changeset
   152
        packages size > 1 ifTrue:[
889d3747a0a8 Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 71
diff changeset
   153
            name :=  packages size printString , ' packages'.
889d3747a0a8 Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 71
diff changeset
   154
        ] ifFalse:[
889d3747a0a8 Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 71
diff changeset
   155
            name :=  packages anElement
889d3747a0a8 Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 71
diff changeset
   156
        ]
889d3747a0a8 Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 71
diff changeset
   157
    ].
71
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   158
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   159
    environment label: name.
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   160
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   161
    "
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   162
        LintReport runPackage:'stx:goodies/monticello'.
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   163
        LintReport runPackage:'stx:libjava'
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   164
    "
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   165
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   166
    "Created: / 04-08-2011 / 14:40:06 / Jan Vrany <jan.vrany@fit.cvut.cz>"
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   167
! !
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   168
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   169
!LintReport methodsFor:'private'!
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   170
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   171
lineAndColumn: aString at: anInteger
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   172
	| line last stream |
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   173
	line := 1.
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   174
	last := 0.
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   175
	stream := aString readStream.
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   176
	[ (stream nextLine isNil or: [ anInteger <= stream position ])
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   177
		ifTrue: [ ^ line @ (anInteger - last) ].
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   178
	last := stream position.
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   179
	line := line + 1 ]
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   180
		repeat
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   181
!
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   182
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   183
sourceFilenameFor: aClass
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   184
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   185
    | fn |
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   186
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   187
    fn := aClass package asString replaceAll:$: with:$_;replaceAll:$/ with:$_.
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   188
    fn := fn , '_' , (aClass asString copyReplaceAll:$: with: $_), '.' , aClass programmingLanguage sourceFileSuffix.
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   189
    
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   190
    ^self encodeFilename: fn.
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   191
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   192
    "
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   193
        Builder::LintReportFormat::CheckStyle basicNew
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   194
            sourceFilenameFor: Class
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   195
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   196
        Builder::LintReportFormat::CheckStyle basicNew
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   197
            sourceFilenameFor: Builder::LintReportFormat
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   198
    "
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   199
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   200
    "Created: / 07-10-2011 / 09:06:00 / Jan Vrany <jan.vrany@fit.cvut.cz>"
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   201
! !
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   202
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   203
!LintReport methodsFor:'running'!
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   204
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   205
runReport
82
e676efa2a286 changed: #runReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 79
diff changeset
   206
    | wasTryLocalSources |
71
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   207
82
e676efa2a286 changed: #runReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 79
diff changeset
   208
    [
e676efa2a286 changed: #runReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 79
diff changeset
   209
        wasTryLocalSources := Class tryLocalSourceFirst.
84
77e12dbe97d3 changed: #runReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 82
diff changeset
   210
        Class tryLocalSourceFirst: true.
82
e676efa2a286 changed: #runReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 79
diff changeset
   211
        SmalllintChecker 
e676efa2a286 changed: #runReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 79
diff changeset
   212
            runRule: (RBCompositeLintRule rules: rules)
e676efa2a286 changed: #runReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 79
diff changeset
   213
            onEnvironment: environment.
71
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   214
82
e676efa2a286 changed: #runReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 79
diff changeset
   215
        (environment classes asSortedCollection: [ :a :b | a name <= b name ])
e676efa2a286 changed: #runReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 79
diff changeset
   216
            do: [ :class | self generateClass: class].
e676efa2a286 changed: #runReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 79
diff changeset
   217
    ] ensure:[
e676efa2a286 changed: #runReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 79
diff changeset
   218
        Class tryLocalSourceFirst: wasTryLocalSources 
e676efa2a286 changed: #runReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 79
diff changeset
   219
    ]
71
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   220
84
77e12dbe97d3 changed: #runReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 82
diff changeset
   221
    "Modified: / 23-01-2012 / 09:04:55 / Jan Vrany <jan.vrany@fit.cvut.cz>"
71
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   222
!
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   223
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   224
setUp
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   225
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   226
    super setUp.
117
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
   227
    rules isNil ifTrue:[
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
   228
        rules := (RBCompositeLintRule rulesFor: RBBasicLintRule)
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
   229
                 reject: [ :each | each class name endsWith: 'SpellingRule' ]
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
   230
    ].
71
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   231
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   232
    "Created: / 04-08-2011 / 14:35:27 / Jan Vrany <jan.vrany@fit.cvut.cz>"
117
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
   233
    "Modified: / 28-02-2013 / 23:18:57 / Jan Vrany <jan.vrany@fit.cvut.cz>"
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
   234
!
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
   235
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
   236
setupRulesFrom: filename
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
   237
    | file |    
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
   238
    file := filename asFilename.
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
   239
    file readingFileDo:[:s|
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
   240
        | spec |
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
   241
        spec := Parser parseLiteralArray: s.
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
   242
        rules := spec decodeAsLiteralArray rules.
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
   243
    ]
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
   244
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
   245
    "Created: / 28-02-2013 / 23:17:41 / Jan Vrany <jan.vrany@fit.cvut.cz>"
71
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   246
! !
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   247
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   248
!LintReport methodsFor:'testing'!
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   249
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   250
isClassEnvironment: anEnvironment
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   251
	^ #(CategoryEnvironment ClassEnvironment VariableEnvironment) includes: anEnvironment class name
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   252
!
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   253
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   254
isSelectorEnvironment: anEnvironment
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   255
	^ #(SelectorEnvironment ParseTreeEnvironment VariableEnvironment) includes: anEnvironment class name
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   256
! !
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   257
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   258
!LintReport class methodsFor:'documentation'!
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   259
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   260
version
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   261
    ^ '$Header$'
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   262
!
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   263
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   264
version_CVS
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   265
    ^ '$Header$'
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   266
!
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   267
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   268
version_SVN
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   269
    ^ '§Id: Builder__LintReport.st 293 2011-11-25 21:42:31Z vranyj1 §'
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   270
! !
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   271
117
d7f87303b984 class: Builder::LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 84
diff changeset
   272
71
1ed7c6cb2f68 initial checkin
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
diff changeset
   273
LintReport initialize!