parkera101093 opened a new pull request, #1296:
URL: https://github.com/apache/daffodil/pull/1296

   argType parameter wasn't needed for FNZeroArgExpr since it's supposed to 
implement dfdl functions that take no argument those functions are 
fn:local-name and fn:namespace-uri functions.
   
   argType is used for targetTypeForSubexpression which should never be called 
in FNZeroArgExpr since their are zero arguments. However, 
targetTypeForSubexpression still has be implemented in the class since 
FNZeroArgExpr extends FunctionCallBase, so targetTypeForSubexpression is set to 
Node.Exists in the FNZeroArgExpr class definition.
   
   In addtion, parameter were removed from FNLocalName0 and FNNamespaceUri0 
since these parameters were no longer needed after FNZeroArgExpr was updated. 
Also, to support this change, both classes now extend RecipeOp instead of 
RecipeOpWithSubRecipes since RecipeOp needs no parameters and the subRecipes 
feature wasn't being used in these functions anyways.
   
    DAFFODIL-2331


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to