[bisq-network/bisq] Adjust lombok annotations to reduce build warnings (#5021)
Stan
notifications at github.com
Tue Dec 29 16:48:58 CET 2020
This change fixes the underlying problems behind five compiler warnings: `Generating equals/hashCode implementation but without a call to superclass`.
Some core classes use `@Value` and `@Data` annotations where not necessary, for example, where `equals`, `hashCode`, and `toString` methods, and `final` class and field modifiers are defined by a developer.
>From lombok classes and documents for the shortcut annotations [`@Value`](https://projectlombok.org/features/Value) and [`@Data`]( https://projectlombok.org/features/Data), we find some implicit annotations applied by these shortcuts are not needed. Adjustments are described below.
- `Tx` Replace `@Value` with `@Getter`.
- `TxOutput` Replace `@Data` with `@Getter` and `@Setter`.
- `ChangeParamProposal` Replace `@Value` with `@Getter`.
- `GetInventoryRequest`, `GetInventoryResponse` An `@EqualsAndHashCode(callSuper = false) `is added to supress build warnings.
You can view, comment on, or merge this pull request online at:
https://github.com/bisq-network/bisq/pull/5021
-- Commit Summary --
* Adjust lombok annotations to reduce build warnings
-- File Changes --
M core/src/main/java/bisq/core/dao/state/model/blockchain/Tx.java (4)
M core/src/main/java/bisq/core/dao/state/model/blockchain/TxOutput.java (6)
M core/src/main/java/bisq/core/dao/state/model/governance/ChangeParamProposal.java (6)
M core/src/main/java/bisq/core/network/p2p/inventory/messages/GetInventoryRequest.java (2)
M core/src/main/java/bisq/core/network/p2p/inventory/messages/GetInventoryResponse.java (2)
-- Patch Links --
https://github.com/bisq-network/bisq/pull/5021.patch
https://github.com/bisq-network/bisq/pull/5021.diff
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/5021
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20201229/76702a07/attachment-0001.htm>
More information about the bisq-github
mailing list