Title: Stabilize the behavior of ConstructorAnalyzer by volivan239 · Pull Request #1293 · UnitTestBot/UTBotJava · GitHub
Open Graph Title: Stabilize the behavior of ConstructorAnalyzer by volivan239 · Pull Request #1293 · UnitTestBot/UTBotJava
X Title: Stabilize the behavior of ConstructorAnalyzer by volivan239 · Pull Request #1293 · UnitTestBot/UTBotJava
Description: Description Removed vistiedConstructors from ConstructorAnalyzer.analyze() method. The intention of that container was to prevent from infinite recursion and to increase performance when analyzing chain of constructors. However, it lead to possible errors if the outer method was called more then once. Also, infinite recursion isn't actually possible since recursive constructor calls are forbidden both in Java and Kotlin. Performance changes seems to be miserable. Type of Change Minor bug fix (non-breaking small changes) How Has This Been Tested? Manual Scenario Tested on the following class -- previously incorrect behavior of ConstructAnalyzer lead to unnecessary reflection used for expected models, after these changes no reflection is used. class NullabilityIssuesExample(val x: Int) { fun copyWithX(new_x: Int?): NullabilityIssuesExample? { if (new_x == null) return null return NullabilityIssuesExample(x + new_x) } } Checklist (remove irrelevant options): This is the author self-check list The change followed the style guidelines of the UTBot project Self-review of the code is passed The change contains enough commentaries, particularly in hard-to-understand areas New documentation is provided or existed one is altered No new warnings New tests have been added All tests pass locally with my changes
Open Graph Description: Description Removed vistiedConstructors from ConstructorAnalyzer.analyze() method. The intention of that container was to prevent from infinite recursion and to increase performance when analyzing ...
X Description: Description Removed vistiedConstructors from ConstructorAnalyzer.analyze() method. The intention of that container was to prevent from infinite recursion and to increase performance when analyzing ...
Opengraph URL: https://github.com/UnitTestBot/UTBotJava/pull/1293
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/checks(.:format) |
| route-controller | pull_requests |
| route-action | checks |
| fetch-nonce | v2:f865f72b-bba0-e0b2-44a3-037eacf9f6f4 |
| current-catalog-service-hash | 87dc3bc62d9b466312751bfd5f889726f4f1337bdff4e8be7da7c93d6c00a25a |
| request-id | E368:13DA52:252324A:2F0A638:69919F4E |
| html-safe-nonce | 2ee69598c457622be2b626460bc0780d71da2eb48c4ba7d1754f2691ee006c0c |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJFMzY4OjEzREE1MjoyNTIzMjRBOjJGMEE2Mzg6Njk5MTlGNEUiLCJ2aXNpdG9yX2lkIjoiNjA2MDM3MzE1OTA4NDkyNDc1MCIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | 2ff478fb7dfc2bbaf6fcc6204bbc4b9657236e41f2ef5e2444c295d577401318 |
| hovercard-subject-tag | pull_request:1108286983 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,checks,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/UnitTestBot/UTBotJava/pull/1293/checks |
| twitter:image | https://avatars.githubusercontent.com/u/65076429?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/65076429?s=400&v=4 |
| og:image:alt | Description Removed vistiedConstructors from ConstructorAnalyzer.analyze() method. The intention of that container was to prevent from infinite recursion and to increase performance when analyzing ... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 42c603b9d642c4a9065a51770f75e5e27132fef0e858607f5c9cb7e422831a7b |
| turbo-cache-control | no-preview |
| go-import | github.com/UnitTestBot/UTBotJava git https://github.com/UnitTestBot/UTBotJava.git |
| octolytics-dimension-user_id | 87413538 |
| octolytics-dimension-user_login | UnitTestBot |
| octolytics-dimension-repository_id | 480810501 |
| octolytics-dimension-repository_nwo | UnitTestBot/UTBotJava |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 480810501 |
| octolytics-dimension-repository_network_root_nwo | UnitTestBot/UTBotJava |
| turbo-body-classes | logged-out env-production page-responsive full-width full-width-p-0 |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 848bc6032dcc93a9a7301dcc3f379a72ba13b96e |
| ui-target | canary-1 |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width