commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Joerg Schaible (JIRA)" <>
Subject [jira] [Commented] (CONFIGURATION-527) clearPropertyDirect() in AbstractConfiguration should be abstract
Date Tue, 26 Feb 2013 13:02:13 GMT


Joerg Schaible commented on CONFIGURATION-527:

IMHO, this will break binary compatibility. Any custom class that did not yet overwrite this
method will now suddenly fail with an Error, if the new version is used as drop-in.
> clearPropertyDirect() in AbstractConfiguration should be abstract
> -----------------------------------------------------------------
>                 Key: CONFIGURATION-527
>                 URL:
>             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:

View raw message