struts-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From josephw <>
Subject [GitHub] struts pull request: WW-4360: Fix a few NullPointerExceptions iden...
Date Wed, 11 Jun 2014 15:05:40 GMT
GitHub user josephw opened a pull request:

     WW-4360: Fix a few NullPointerExceptions identified by Eclipse #15 

    Eclipse's static analysis of nulls identifies a number of spots where a NullPointerException
can occur. This change fixes them either to rearrange things so they can't happen or to throw
a more specific exception.

You can merge this pull request into a Git repository by running:

    $ git pull master

Alternatively you can review and apply these changes as the patch at:

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #16
commit 81cdd6e0d8fe01616e5c3055e5536ee63199b6e6
Author: Joseph Walton <>
Date:   2014-06-10T12:21:39Z

    Clean up potential NullPointerExceptions.
    - Construct BufferedReaders ahead of try blocks
    - Don't catch exceptions in tests
    - Specifically check for mismatched braces in patterns
    - Fail on an unknown numeric type

commit f98e92c154add1c869efe88ce1fdba984a5f71aa
Author: Joseph Walton <>
Date:   2014-06-10T12:21:55Z

    Fix potential NullPointerExceptions.
    - Allow for a null location when logging
    - Don't try to use a null LocationFinder


If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at or file a JIRA ticket
with INFRA.

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message