I agree.

Found in FOP only 1 method witch javadoc explicitely says that is not
part of external API
(see o.a.fop.apps.FopFactory.java#getColorSpaceCache())

Perhaps this should be extended to the whole public material that should
only be invoked as internal.

Le 24/01/2012 12:16, Chris Bowditch a écrit :
> On 23/01/2012 22:41, J.Pietschmann wrote:
>> Hi Mehdi!
> Hi Joerg, Mehdi,
>> Am 23.01.2012 17:15, schrieb me...@apache.org:
>>> This breaks the public API but for good reasons:
>> Good reasons or not, there are a few guidelines when it comes
>> to changing public interfaces:
>> - There has to be a discussion of the change. Alternatively,
>>    open a bugzilla ticket and provide reasons and the proposed
>>    solution/patch for inspection (yes, even committers should
>>    do that)
>> - It's good practice to call a vote after the pros and cons
>>    of the change have been laid out.
>> - It's also good practice to deprecate functionality which is
>>    going to be removed, in order to maintain compatibility for
>>    at least one release cycle and give users time to adapt.
>> Please revert the commit.
> I agree this should have been discussed before it was committed. Mehdi 
> has now reverted the change and put the changes into a patch. Most of 
> the Java code in FOP has a public modifier, but that in my view does not 
> mean it is part of the public API for embedding FOP. I think we need a 
> clearer definition of what is part of the public API and what is not and 
> we should document this.
> Thanks,
> Chris
>> J.Pietschmann


Reply via email to