Home - News ( United States | United Kingdom | Italy | Germany ) - Football scores

Showing content from https://github.com/pmd/pmd/issues/1307 below:

[java] AccessNode API changes · Issue #1307 · pmd/pmd · GitHub

The purpose of AccessNode is to denote a node which can have declaration modifiers, eg abstract, final, public, etc. Some nodes implement it just to get the "final" modifier:

• ASTFormalParameter
• ASTResource (relates to [java] AST inconsistencies around FormalParameter #998)
• ASTLocalVariableDeclaration

ASTLambdaExpression implements it for nothing, due to an implementation error: AbstractMethodLikeNode extends AbstractJavaAccessNode...

1. I think all of these should stop implementing the interface in 7.0.0. We could have a separate base class for nodes that can be final, or implement it ad-hoc with a boolean since FormalParameter is a TypeNode and so we'd have to have several base classes. The amount of duplication seems acceptable
2. The setters should be removed from the interface. They're only used by the parser, and could be package-private

RetroSearch is an open source project built by @garambo | Open a GitHub Issue

Search and Browse the WWW like it's 1997 | Search results from DuckDuckGo

HTML: 3.2 | Encoding: UTF-8 | Version: 0.7.4