False positive when making a package-private class public that already has a public static method's different return type about japicmp

[siom79]

Changing the return type from an interface to a concrete implementation causes the incompatibility METHOD_RETURN_TYPE_CHANGED as now source code assigning the return value to another implementation of the interface won't work any longer.
This is already detected in the first run of japicmp, but the class is package protected and therefore filtered.
If you change the configuration of japicmp to package_protected, you will see the change in the first run:

<plugin>
        <groupId>com.github.siom79.japicmp</groupId>
        <artifactId>japicmp-maven-plugin</artifactId>
        <configuration>
          <parameter>
            <accessModifier>package_protected</accessModifier>
          </parameter>
        </configuration>
      </plugin>

PS: The documentation is currently wrong, as the option is named there protected and not package_protected.

[garydgregory]

Uh? Please help me understand: If a released version of a library contains a package private class, then that class is NOT accessible to the outside world. So how can there be a break in compatibility for clients in the next version when that class is made public, no matter what changes?

[siom79]

You are right. When the class changes to public, all incompatibilities should be ignored.

[garydgregory]

You are right. When the class changes to public, all incompatibilities should be ignored.

Ah, thank you 😄 😄

[siom79]

Fixed with this commit.