Skip to content

Commit e349eec

Browse files
idodeclareidodeclare
committed
Add VBXrefTest with one non-passing for dangling span
- Fix keyword resource bug by defining inline in Consts instead. (VB is case-insensitive -- that bug will be fixed in subsequent commits.)
1 parent 1c30da9 commit e349eec

File tree

9 files changed

+1209
-175
lines changed

9 files changed

+1209
-175
lines changed

‎opengrok-indexer/pom.xml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -158,6 +158,13 @@ Portions Copyright (c) 2017, Chris Fraire <[email protected]>.
158158
<exclude>*.java</exclude>
159159
</excludes>
160160
</testResource>
161+
<testResource>
162+
<targetPath>org/opensolaris/opengrok/analysis/vb/</targetPath>
163+
<directory>../test/org/opensolaris/opengrok/analysis/vb/</directory>
164+
<excludes>
165+
<exclude>*.java</exclude>
166+
</excludes>
167+
</testResource>
161168
</testResources>
162169

163170
<plugins>

‎src/org/opensolaris/opengrok/analysis/vb/Consts.java

Lines changed: 154 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -19,15 +19,12 @@
1919

2020
/*
2121
* Copyright (c) 2007, 2013, Oracle and/or its affiliates. All rights reserved.
22+
* Portions Copyright (c) 2017, Chris Fraire <[email protected]>.
2223
*/
2324
package org.opensolaris.opengrok.analysis.vb;
2425

25-
import java.io.BufferedReader;
26-
import java.io.IOException;
27-
import java.io.InputStreamReader;
2826
import java.util.Collections;
2927
import java.util.HashSet;
30-
import java.util.Locale;
3128
import java.util.Set;
3229

3330
@SuppressWarnings("PMD.AvoidThrowingRawExceptionTypes")
@@ -37,33 +34,166 @@ public final class Consts {
3734

3835
static {
3936
HashSet<String> kwds = new HashSet<String>();
40-
try {
41-
42-
populateKeywordSet(kwds, "vbReservedWords.dat");
43-
} catch (IOException ioe) {
44-
throw new RuntimeException(ioe);
45-
}
37+
populateKeywordSet(kwds);
4638
reservedKeywords = Collections.unmodifiableSet(kwds);
4739
}
4840

4941
private Consts() {
5042
// Util class, can not be constructed.
5143
}
5244

53-
private static void populateKeywordSet(Set<String> set, String file)
54-
throws IOException {
55-
String line, lline;
56-
try (BufferedReader reader = new BufferedReader(new InputStreamReader(
57-
Consts.class.getResourceAsStream(file), "US-ASCII"))) {
58-
while ((line = reader.readLine()) != null) {
59-
line = line.trim();
60-
lline = line.toLowerCase(Locale.US);
61-
if (line.charAt(0) != '#') {
62-
set.add(line);
63-
set.add(lline);
64-
}
65-
}
66-
}
45+
private static void populateKeywordSet(Set<String> kwd) {
46+
kwd.add("AddHandler");
47+
kwd.add("AddressOf");
48+
kwd.add("Alias");
49+
kwd.add("And");
50+
kwd.add("AndAlso");
51+
kwd.add("As");
52+
kwd.add("Boolean");
53+
kwd.add("ByRef");
54+
kwd.add("Byte");
55+
kwd.add("ByVal");
56+
kwd.add("Call");
57+
kwd.add("Case");
58+
kwd.add("Catch");
59+
kwd.add("CBool");
60+
kwd.add("CByte");
61+
kwd.add("CChar");
62+
kwd.add("CDate");
63+
kwd.add("CDec");
64+
kwd.add("CDbl");
65+
kwd.add("Char");
66+
kwd.add("CInt");
67+
kwd.add("Class");
68+
kwd.add("CLng");
69+
kwd.add("CObj");
70+
kwd.add("Const");
71+
kwd.add("Continue");
72+
kwd.add("CSByte");
73+
kwd.add("CShort");
74+
kwd.add("CSng");
75+
kwd.add("CStr");
76+
kwd.add("CType");
77+
kwd.add("CUInt");
78+
kwd.add("CULng");
79+
kwd.add("CUShort");
80+
kwd.add("Date");
81+
kwd.add("Decimal");
82+
kwd.add("Declare");
83+
kwd.add("Default");
84+
kwd.add("Delegate");
85+
kwd.add("Dim");
86+
kwd.add("DirectCast");
87+
kwd.add("Do");
88+
kwd.add("Double");
89+
kwd.add("Each");
90+
kwd.add("Else");
91+
kwd.add("ElseIf");
92+
kwd.add("End");
93+
kwd.add("EndIf");
94+
kwd.add("Enum");
95+
kwd.add("Erase");
96+
kwd.add("Error");
97+
kwd.add("Event");
98+
kwd.add("Exit");
99+
kwd.add("False");
100+
kwd.add("Finally");
101+
kwd.add("For");
102+
kwd.add("Friend");
103+
kwd.add("Function");
104+
kwd.add("Get");
105+
kwd.add("GetType");
106+
kwd.add("Global");
107+
kwd.add("GoSub");
108+
kwd.add("GoTo");
109+
kwd.add("Handles");
110+
kwd.add("If");
111+
kwd.add("Implements");
112+
kwd.add("Imports");
113+
kwd.add("In");
114+
kwd.add("Inherits");
115+
kwd.add("Integer");
116+
kwd.add("Interface");
117+
kwd.add("Is");
118+
kwd.add("IsNot");
119+
kwd.add("Let");
120+
kwd.add("Lib");
121+
kwd.add("Like");
122+
kwd.add("Long");
123+
kwd.add("Loop");
124+
kwd.add("Me");
125+
kwd.add("Mod");
126+
kwd.add("Module");
127+
kwd.add("MustInherit");
128+
kwd.add("MustOverride");
129+
kwd.add("MyBase");
130+
kwd.add("MyClass");
131+
kwd.add("Namespace");
132+
kwd.add("Narrowing");
133+
kwd.add("New");
134+
kwd.add("Next");
135+
kwd.add("Not");
136+
kwd.add("Nothing");
137+
kwd.add("NotInheritable");
138+
kwd.add("NotOverridable");
139+
kwd.add("Object");
140+
kwd.add("Of");
141+
kwd.add("On");
142+
kwd.add("Operator");
143+
kwd.add("Option");
144+
kwd.add("Optional");
145+
kwd.add("Or");
146+
kwd.add("OrElse");
147+
kwd.add("Overloads");
148+
kwd.add("Overridable");
149+
kwd.add("Overrides");
150+
kwd.add("ParamArray");
151+
kwd.add("Partial");
152+
kwd.add("Private");
153+
kwd.add("Property");
154+
kwd.add("Protected");
155+
kwd.add("Public");
156+
kwd.add("RaiseEvent");
157+
kwd.add("ReadOnly");
158+
kwd.add("ReDim");
159+
kwd.add("REM");
160+
kwd.add("RemoveHandler");
161+
kwd.add("Resume");
162+
kwd.add("Return");
163+
kwd.add("SByte");
164+
kwd.add("Select");
165+
kwd.add("Set");
166+
kwd.add("Shadows");
167+
kwd.add("Shared");
168+
kwd.add("Short");
169+
kwd.add("Single");
170+
kwd.add("Static");
171+
kwd.add("Step");
172+
kwd.add("Stop");
173+
kwd.add("String");
174+
kwd.add("Structure");
175+
kwd.add("Sub");
176+
kwd.add("SyncLock");
177+
kwd.add("Then");
178+
kwd.add("Throw");
179+
kwd.add("To");
180+
kwd.add("True");
181+
kwd.add("Try");
182+
kwd.add("TryCast");
183+
kwd.add("TypeOf");
184+
kwd.add("Variant");
185+
kwd.add("Wend");
186+
kwd.add("UInteger");
187+
kwd.add("ULong");
188+
kwd.add("UShort");
189+
kwd.add("Using");
190+
kwd.add("When");
191+
kwd.add("While");
192+
kwd.add("Widening");
193+
kwd.add("With");
194+
kwd.add("WithEvents");
195+
kwd.add("WriteOnly");
196+
kwd.add("Xor");
67197
}
68198

69199
static Set<String> getReservedKeywords() {

‎src/org/opensolaris/opengrok/analysis/vb/vbReservedWords.dat

Lines changed: 0 additions & 151 deletions
This file was deleted.

0 commit comments

Comments
 (0)