RFR: 8327791: Optimization for new BigDecimal(String) [v16]

Chen Liang liach at openjdk.org
Thu Mar 21 02:16:27 UTC 2024


On Thu, 21 Mar 2024 00:40:45 GMT, Shaojin Wen <duke at openjdk.org> wrote:

> Should we declare the BigDecimal(CharSequence,MathContext) method as public? Scenarios like [MysqlBinaryValueDecoder#decodeDecimal](https://github.com/mysql/mysql-connector-j/blob/release/8.x/src/main/protocol-impl/java/com/mysql/cj/protocol/a/MysqlBinaryValueDecoder.java#L275) can be used to avoid a memory allocation.

Good idea, but since this involves a change in the public APIs and needs extra care with API specification, you should do this in a dependent PR (target the pr/18177 branch of jdk repo) that focus specifically on the addition. A new API is out of scope for this current PR.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/18177#issuecomment-2011064004


More information about the core-libs-dev mailing list