RFR: 2279: Improve JSONNumber::asInt usage to prevent future break
Erik Joelsson
erikj at openjdk.org
Thu Jun 6 21:45:43 UTC 2024
On Thu, 6 Jun 2024 21:24:09 GMT, Zhao Song <zsong at openjdk.org> wrote:
> [SKARA-2277](https://bugs.openjdk.org/browse/SKARA-2277) is an instance of a breakage caused by improper usage of JSONNumber::asInt. In our project, there are other usages of JSONNumber::asInt, we should clean them up to prevent future break.
>
> As Erik said, he thinks it's fine to treat user ids and repository ids as int.
>
> Here are the other 2 places that might break.
>
> GitLabRepository::toCommitComment treats note ids as int.
> GitHubPullRequest::reviews treats review ids as int.
>
> Besides, I changed the type of id from int to String in Review.java
Marked as reviewed by erikj (Lead).
-------------
PR Review: https://git.openjdk.org/skara/pull/1657#pullrequestreview-2103266465
More information about the skara-dev
mailing list