Skip to content

Conversation

gavinking
Copy link
Member

@gavinking gavinking commented Sep 27, 2025

Redo #10249.

[Please describe here what your change is about]


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.


@hibernate-github-bot
Copy link

hibernate-github-bot bot commented Sep 27, 2025

Thanks for your pull request!

This pull request does not follow the contribution rules. Could you have a look?

❌ All commit messages should start with a JIRA issue key matching pattern HHH-\d+
    ↳ Offending commits: [754d192]

› This message was automatically generated.

@gavinking gavinking force-pushed the rowtypejdbcmapping-2 branch 3 times, most recently from 60fba3d to 9e1f71e Compare September 27, 2025 20:21
// TODO: appliedBinding can be null here, resulting in NPE
if ( binding == null || !areEqualBindings( appliedBinding, binding ) ) {
if ( binding == null
|| !equal( appliedBinding, binding, appliedBinding.getBindType().getJavaTypeDescriptor() ) ) {

Check warning

Code scanning / CodeQL

Dereferenced variable may be null Warning

Variable
appliedBinding
may be null at this access as suggested by
this
null guard.
@gavinking gavinking marked this pull request as ready for review September 27, 2025 20:55
@gavinking gavinking merged commit 0f41601 into hibernate:main Sep 27, 2025
24 of 25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant