GitHub user BruceKuiLiu opened a pull request:

    https://github.com/apache/ant/pull/45

    Consider returning a zero length array rather than null.

    It is often a better design to return a length zero array rather than a 
null reference to indicate that there are no results (i.e., an empty list of 
results).
    This way, no explicit check for null is needed by clients of the method.
    On the other hand, using null to indicate "there is no answer to this 
question" is probably appropriate.
    
http://findbugs.sourceforge.net/bugDescriptions.html#PZLA_PREFER_ZERO_LENGTH_ARRAYS

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

    $ git pull https://github.com/BruceKuiLiu/ant master6

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

    https://github.com/apache/ant/pull/45.patch

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

    This closes #45
    
----
commit ca2fbf36183b82457fe6d161b77a6b10ae2e457f
Author: Kui LIU <brucekui...@gmail.com>
Date:   2017-10-14T15:06:08Z

    Consider returning a zero length array rather than null.
    
    It is often a better design to return a length zero array rather than a 
null reference to indicate that there are no results (i.e., an empty list of 
results).
    This way, no explicit check for null is needed by clients of the method.
    On the other hand, using null to indicate "there is no answer to this 
question" is probably appropriate.
    
http://findbugs.sourceforge.net/bugDescriptions.html#PZLA_PREFER_ZERO_LENGTH_ARRAYS

----


---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org
For additional commands, e-mail: dev-h...@ant.apache.org

Reply via email to