[
https://issues.apache.org/jira/browse/CONFIGURATION-527?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13587113#comment-13587113
]
Matthias Richter commented on CONFIGURATION-527:
------------------------------------------------
That's fine for me. I just stumbled over it when implementing the
AbstractConfiguration.
The patch file attached is fine, though?
> clearPropertyDirect() in AbstractConfiguration should be abstract
> -----------------------------------------------------------------
>
> Key: CONFIGURATION-527
> URL: https://issues.apache.org/jira/browse/CONFIGURATION-527
> Project: Commons Configuration
> Issue Type: Improvement
> Components: Build
> Affects Versions: 1.9
> Reporter: Matthias Richter
> Priority: Minor
> Attachments: patch.diff
>
>
> The method clearPropertyDirect() in AbstractConfiguration has an empty
> implementation body. It is documented as "should be overwritten" but is not
> marked abstract.
> It should be abstract to prevent errors when subclassing
> AbstractConfiguration and using the clear() method where (indirectly)
> clearPropertyDirect() is being called.
> See addPropertyDirect() for comparison, which is abstract.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
