|
|
To be able to use findbugs to weed out problems in the HTML parser, we
need to tighten down its dependencies. The lexer package is one of
these.
I added a package declaration for that package with a
@ParametersAreNonnullByDefault annotation, then ran findbugs and
whacked the null problems in that package.
This change also includes a bit of opportunistic cleanup of other
findbugs problems.
Submitted @4328
Total comments: 1
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+176 lines, -107 lines) |
Patch |
 |
M |
build.xml
|
View
|
|
1 chunk |
+1 line, -1 line |
1 comment
|
Download
|
 |
M |
src/com/google/caja/ancillary/jsdoc/AnnotationHandlers.java
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
src/com/google/caja/ancillary/jsdoc/JsdocMain.java
|
View
|
|
1 chunk |
+9 lines, -5 lines |
0 comments
|
Download
|
 |
M |
src/com/google/caja/ancillary/jsdoc/UpdocParser.java
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
src/com/google/caja/ancillary/opt/ParseTreeKB.java
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
src/com/google/caja/ancillary/opt/StatementSimplifier.java
|
View
|
|
2 chunks |
+4 lines, -4 lines |
0 comments
|
Download
|
 |
M |
src/com/google/caja/ancillary/servlet/CajaWebToolsServlet.java
|
View
|
|
1 chunk |
+3 lines, -1 line |
0 comments
|
Download
|
 |
M |
src/com/google/caja/demos/playground/client/ui/PlaygroundView.java
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
src/com/google/caja/lexer/CharProducer.java
|
View
|
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
 |
M |
src/com/google/caja/lexer/ExternalReference.java
|
View
|
|
2 chunks |
+3 lines, -1 line |
0 comments
|
Download
|
 |
M |
src/com/google/caja/lexer/FilePosition.java
|
View
|
|
3 chunks |
+4 lines, -2 lines |
0 comments
|
Download
|
 |
M |
src/com/google/caja/lexer/GuessContentType.java
|
View
|
|
2 chunks |
+4 lines, -1 line |
0 comments
|
Download
|
 |
M |
src/com/google/caja/lexer/InputSource.java
|
View
|
|
2 chunks |
+3 lines, -1 line |
0 comments
|
Download
|
 |
M |
src/com/google/caja/lexer/PositionInferer.java
|
View
|
|
4 chunks |
+6 lines, -4 lines |
0 comments
|
Download
|
 |
M |
src/com/google/caja/lexer/TokenConsumer.java
|
View
|
|
2 chunks |
+4 lines, -2 lines |
0 comments
|
Download
|
 |
A |
src/com/google/caja/lexer/package-info.java
|
View
|
|
1 chunk |
+6 lines, -0 lines |
0 comments
|
Download
|
 |
M |
src/com/google/caja/parser/css/CssParser.java
|
View
|
|
1 chunk |
+4 lines, -0 lines |
0 comments
|
Download
|
 |
M |
src/com/google/caja/parser/css/CssTree.java
|
View
|
|
2 chunks |
+4 lines, -1 line |
0 comments
|
Download
|
 |
M |
src/com/google/caja/parser/html/Nodes.java
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
src/com/google/caja/parser/js/AbstractExpression.java
|
View
|
|
2 chunks |
+3 lines, -1 line |
0 comments
|
Download
|
 |
M |
src/com/google/caja/parser/js/Expression.java
|
View
|
|
2 chunks |
+3 lines, -1 line |
0 comments
|
Download
|
 |
M |
src/com/google/caja/parser/js/Operation.java
|
View
|
|
2 chunks |
+3 lines, -1 line |
0 comments
|
Download
|
 |
M |
src/com/google/caja/parser/quasiliteral/DefaultValijaRewriter.java
|
View
|
|
1 chunk |
+3 lines, -3 lines |
0 comments
|
Download
|
 |
M |
src/com/google/caja/plugin/CssRuleRewriter.java
|
View
|
|
2 chunks |
+5 lines, -3 lines |
0 comments
|
Download
|
 |
M |
src/com/google/caja/plugin/DataUriFetcher.java
|
View
|
|
1 chunk |
+3 lines, -4 lines |
0 comments
|
Download
|
 |
M |
src/com/google/caja/plugin/Planner.java
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
src/com/google/caja/render/BufferingRenderer.java
|
View
|
|
4 chunks |
+9 lines, -6 lines |
0 comments
|
Download
|
 |
M |
src/com/google/caja/render/Concatenator.java
|
View
|
|
2 chunks |
+3 lines, -1 line |
0 comments
|
Download
|
 |
M |
src/com/google/caja/render/CssMinimalPrinter.java
|
View
|
|
2 chunks |
+3 lines, -1 line |
0 comments
|
Download
|
 |
M |
src/com/google/caja/render/CssPrettyPrinter.java
|
View
|
|
3 chunks |
+5 lines, -3 lines |
0 comments
|
Download
|
 |
M |
src/com/google/caja/render/JsLinePreservingPrinter.java
|
View
|
|
2 chunks |
+4 lines, -2 lines |
0 comments
|
Download
|
 |
M |
src/com/google/caja/render/JsMinimalPrinter.java
|
View
|
|
2 chunks |
+3 lines, -2 lines |
0 comments
|
Download
|
 |
M |
src/com/google/caja/render/SideBySideRenderer.java
|
View
|
|
4 chunks |
+9 lines, -8 lines |
0 comments
|
Download
|
 |
M |
src/com/google/caja/render/SourceSnippetRenderer.java
|
View
|
|
3 chunks |
+4 lines, -2 lines |
0 comments
|
Download
|
 |
M |
src/com/google/caja/render/SourceSpansRenderer.java
|
View
|
|
2 chunks |
+3 lines, -1 line |
0 comments
|
Download
|
 |
M |
src/com/google/caja/render/TokenClassification.java
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
src/com/google/caja/reporting/MessagePart.java
|
View
|
|
2 chunks |
+4 lines, -1 line |
0 comments
|
Download
|
 |
M |
src/com/google/caja/util/Multimap.java
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
src/com/google/caja/util/RhinoExecutor.java
|
View
|
|
1 chunk |
+10 lines, -4 lines |
0 comments
|
Download
|
 |
M |
tests/com/google/caja/ancillary/jsdoc/AnnotationHandlersTest.java
|
View
|
|
1 chunk |
+1 line, -0 lines |
0 comments
|
Download
|
 |
M |
tests/com/google/caja/lexer/FetchedDataTest.java
|
View
|
|
4 chunks |
+6 lines, -3 lines |
0 comments
|
Download
|
 |
M |
tests/com/google/caja/lexer/PositionInfererTest.java
|
View
|
|
3 chunks |
+4 lines, -2 lines |
0 comments
|
Download
|
 |
M |
tests/com/google/caja/parser/js/JsFunFuzz.java
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
tests/com/google/caja/parser/quasiliteral/MatchExperiments.java
|
View
|
|
2 chunks |
+7 lines, -9 lines |
0 comments
|
Download
|
 |
M |
tests/com/google/caja/parser/quasiliteral/ModuleFormatTest.java
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
tests/com/google/caja/parser/quasiliteral/RewriterTestCase.java
|
View
|
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
 |
M |
tests/com/google/caja/plugin/BrowserTestCase.java
|
View
|
|
5 chunks |
+9 lines, -9 lines |
0 comments
|
Download
|
 |
M |
tests/com/google/caja/plugin/templates/TemplateCompilerTest.java
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
tests/com/google/caja/render/SourceSpansRendererTest.java
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
tests/com/google/caja/service/TestHttpServletRequest.java
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
Total messages: 4
|