siom79 / japicmp

Comparison of two versions of a jar archive
https://siom79.github.io/japicmp
Apache License 2.0
694 stars 107 forks source link

Take `volatile` modifier into account #378

Closed guillermocalvo closed 5 months ago

guillermocalvo commented 5 months ago

Currently, volatile modifier is not taken into account.

It's not included when retrieving field modifiers. In addition, changes to this modifier are not reported.

As per the specs, binary compatibility is not affected by changes to a field's volatile modifier.

However, they can have a big semantic impact in terms of synchronization and the memory model. Therefore, they should at least be reported.