On 2015-02-16, Harald Brabenetz wrote: > I would like to make the CompareMatcher extendable. This means that > the constructor and two methods must be protected instead private.
The two protected methods would be the checkFor methods? You need those so you can have your own factory methods, AFAIU. If this was the only usecase they could be final. > This example is basically what I do with the Matcher on my current > Project: I write the test-Input into the FileSystem, where I can > manual review it. And when the changes are OK I can simply overwrite > my control-File with the written Test-File. You are cheating ;-) > On the other hand it is more difficult to maintain a Class which can > be extended. A protected constructor will signal "this class is extension friendly" so we better review its API and implementation to see whether this really is the case. Other than that, I don't see any issue. Stefan ------------------------------------------------------------------------------ Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server from Actuate! Instantly Supercharge Your Business Reports and Dashboards with Interactivity, Sharing, Native Excel Exports, App Integration & more Get technology previously reserved for billion-dollar corporations, FREE http://pubads.g.doubleclick.net/gampad/clk?id=190641631&iu=/4140/ostg.clktrk _______________________________________________ Xmlunit-general mailing list Xmlunit-general@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/xmlunit-general