Skip to content

Conversation

gavinking
Copy link
Member

[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 May 30, 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: [3232d98]

› This message was automatically generated.

return bits.length > 2 ? Integer.parseInt( bits[2] ) : 0;
else {
final String[] bits = split( ". -", databaseVersion );
return bits.length > 2 ? Integer.parseInt( bits[2] ) : 0;

Check notice

Code scanning / CodeQL

Missing catch of NumberFormatException

Potential uncaught 'java.lang.NumberFormatException'.
// 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

Variable [appliedBinding](1) may be null at this access as suggested by [this](2) null guard.
@gavinking gavinking force-pushed the rowtypejdbcmapping branch 2 times, most recently from eb5b9c5 to eba5208 Compare June 1, 2025 19:55
@gavinking gavinking force-pushed the rowtypejdbcmapping branch from eba5208 to 3232d98 Compare June 1, 2025 21:07
@gavinking gavinking changed the title two cleanups don't use a raw type in JdbcMapping Jun 13, 2025
@gavinking
Copy link
Member Author

Now #11023

@gavinking gavinking closed this Sep 27, 2025
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