Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(33)

Issue 2864041: Add @Nonnull and @Nullable annotations to the lexer package. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
15 years, 5 months ago by MikeSamuel
Modified:
15 years, 5 months ago
Reviewers:
Jasvir
CC:
google-caja-discuss_googlegroups.com
Base URL:
http://google-caja.googlecode.com/svn/trunk/
Visibility:
Public.

Description

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

Patch Set 1 #

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

Messages

Total messages: 4
MikeSamuel
15 years, 5 months ago (2010-11-03 17:52:18 UTC) #1
MikeSamuel
http://codereview.appspot.com/2864041/diff/1/build.xml File build.xml (right): http://codereview.appspot.com/2864041/diff/1/build.xml#newcode82 build.xml:82: <pathelement path="${third_party}/java/jsr305/jsr305.jar"/> So javax.annotations can be used in tests.
15 years, 5 months ago (2010-11-03 17:55:09 UTC) #2
MikeSamuel
ping
15 years, 5 months ago (2010-11-12 18:24:03 UTC) #3
Jasvir
15 years, 5 months ago (2010-11-18 18:42:44 UTC) #4
LGTM
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b