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