fury icon indicating copy to clipboard operation
fury copied to clipboard
verified publisher icon apache

chore(java): Remove redundant code

Open LiangliangSui opened this issue 5 months ago • 2 comments

What does this PR do?

Related issues

Does this PR introduce any user-facing change?

  • [ ] Does this PR introduce any public API change?
  • [ ] Does this PR introduce any binary protocol compatibility change?

Benchmark

LiangliangSui avatar Jul 04 '25 03:07 LiangliangSui

Does readNullable also need to be manually inline?

https://github.com/apache/fory/blob/a9e5d999f58b2a0ec8f22ead9a20040d45e39a73/java/fory-core/src/main/java/org/apache/fory/serializer/SerializationBinding.java#L381-L384

LiangliangSui avatar Jul 04 '25 03:07 LiangliangSui

Does readNullable also need to be manually inline?

https://github.com/apache/fory/blob/a9e5d999f58b2a0ec8f22ead9a20040d45e39a73/java/fory-core/src/main/java/org/apache/fory/serializer/SerializationBinding.java#L381-L384

Yes, we'd better do that

chaokunyang avatar Jul 06 '25 08:07 chaokunyang