[ 
https://issues.apache.org/jira/browse/XALANJ-2556?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13253039#comment-13253039
 ] 

SHIN HWEI TAN commented on XALANJ-2556:
---------------------------------------

The Javadoc comments reported above are only referring to "@param transformer 
non-null reference to the the current transform-time state." for the method 
execute(TransformerImpl transformer) in the templates/ElemFallback.java file 
because the method is empty. 
                
> Inconsistent Javadoc comment and code for execute(TransformerImpl) in 
> org.apache.xalan.templates.ElemFallback
> -------------------------------------------------------------------------------------------------------------
>
>                 Key: XALANJ-2556
>                 URL: https://issues.apache.org/jira/browse/XALANJ-2556
>             Project: XalanJ2
>          Issue Type: Bug
>      Security Level: No security risk; visible to anyone(Ordinary problems in 
> Xalan projects.  Anybody can view the issue.) 
>    Affects Versions: 2.7.1, 2.7
>         Environment: Platform Independent
>            Reporter: SHIN HWEI TAN
>              Labels: javadoc
>   Original Estimate: 0.05h
>  Remaining Estimate: 0.05h
>
> The Javadoc comment below states that the parameter transformer is a 
> "non-null reference to the the current transform-time state.":
>  /**
>   ...
>   * @param transformer non-null reference to the the current transform-time 
> state.
>   ...
>   */
> public void execute(TransformerImpl transformer) throws TransformerException
> However, the method does not check the parameter for "null". It executes 
> normally without throwing any exception when called with a null transformer. 
> Suggested Fixes:
> 1. Add parameter check for "null"
> or
> 2. Remove "non-null" from @param.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

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

Reply via email to