> On Jan 28, 2019, at 2:35 PM, sebb <[email protected]> wrote:
>
> On Mon, 28 Jan 2019 at 19:22, Marcelo Vanzin
> <[email protected]> wrote:
>>
>> Haven't looked at the code, but if it's being compiled for java 8, and
>> the new methods have a default implementation, then it's fine. clirr
>> just complains because it's too old to know about default methods.
>
> I don't think so.
>
> I think Clirr is complaining because it affects source compatibility,
> and the Maven report does not distinguish source/binary complaints.
>
By the way Japicmp says the same thing...that methods were added to interfaces.
>>> On Mon, Jan 28, 2019 at 11:18 AM Rob Tompkins <[email protected]> wrote:
>>>
>>> @Marcelo - Many thanks...Yes. That makes sense. Thanks. Seems like this
>>> release should be a -1 then because we’re breaking BC without a major
>>> version change. Right??
>>>
>>> -Rob
>>>
>>>
>>>> On Jan 28, 2019, at 2:07 PM, Marcelo Vanzin <[email protected]>
>>>> wrote:
>>>>
>>>> On Mon, Jan 28, 2019 at 11:01 AM Rob Tompkins <[email protected]> wrote:
>>>>> Before I vote on the the thread, does adding a method to an interface
>>>>> cause BC to break? I would think not. All of the clirr errors are merely
>>>>> additions. Further the JAPICMP report confirms this.
>>>>
>>>> Existing classes that implement the interface won't have the newly
>>>> added method. So if some other code calls that method in one of these
>>>> implementations, they'll get an "AbstractMethodError" (or something
>>>> similar).
>>>>
>>>> --
>>>> Marcelo
>>>>
>>>> ---------------------------------------------------------------------
>>>> To unsubscribe, e-mail: [email protected]
>>>> For additional commands, e-mail: [email protected]
>>>>
>>>
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: [email protected]
>>> For additional commands, e-mail: [email protected]
>>>
>>
>>
>> --
>> Marcelo
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: [email protected]
>> For additional commands, e-mail: [email protected]
>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [email protected]
> For additional commands, e-mail: [email protected]
>
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
