Hibernate/hibernate-orm
1
0
Fork 0
mirror of https://github.com/hibernate/hibernate-orm synced 2025-02-10 05:04:52 +00:00
Code Issues Packages Projects Releases Wiki Activity

HHH-16711 Ensure static accessor methods for accessing foreign package state use visible types

Browse Source
This commit is contained in:
Christian Beikov 2023-06-20 19:48:30 +02:00
parent d19a6cca03
commit fa3b2e82fb
1 changed files with 22 additions and 35 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

57
hibernate-core/src/main/java/org/hibernate/bytecode/internal/bytebuddy/BytecodeProviderImpl.java
View File

@ -324,23 +324,14 @@ private Class<?> determineAccessOptimizerSuperClass(Class<?> clazz, Member[] get
getterType = ( (Method) getter ).getReturnType();
}
builder = builder.define(
new MethodDescription.InDefinedShape.Latent(
builder.toTypeDescription(),
new MethodDescription.Token(
"get_" + getter.getName(),
Opcodes.ACC_PROTECTED | Opcodes.ACC_STATIC,
TypeDescription.Generic.OfNonGenericType.ForLoadedType.of(
getterType
),
Collections.singletonList(
TypeDescription.Generic.OfNonGenericType.ForLoadedType.of(
clazz
)
)
)
)
builder = builder.defineMethod(
"get_" + getter.getName(),
TypeDescription.Generic.OfNonGenericType.ForLoadedType.of(
getterType
),
Opcodes.ACC_PROTECTED | Opcodes.ACC_STATIC
)
.withParameter( foreignPackageClassInfo.clazz )
.intercept(
new Implementation.Simple(
new GetFieldOnArgument(
@ -358,24 +349,13 @@ private Class<?> determineAccessOptimizerSuperClass(Class<?> clazz, Member[] get
setterType = ( (Method) setter ).getParameterTypes()[0];
}
builder = builder.define(
new MethodDescription.InDefinedShape.Latent(
builder.toTypeDescription(),
new MethodDescription.Token(
"set_" + setter.getName(),
Opcodes.ACC_PROTECTED | Opcodes.ACC_STATIC,
TypeDescription.Generic.VOID,
Arrays.asList(
TypeDescription.Generic.OfNonGenericType.ForLoadedType.of(
clazz
),
TypeDescription.Generic.OfNonGenericType.ForLoadedType.of(
setterType
)
)
)
)
builder = builder.defineMethod(
"set_" + setter.getName(),
TypeDescription.Generic.VOID,
Opcodes.ACC_PROTECTED | Opcodes.ACC_STATIC
)
.withParameter( foreignPackageClassInfo.clazz )
.withParameter( setterType )
.intercept(
new Implementation.Simple(
new SetFieldOnArgument(
@ -781,7 +761,10 @@ else if ( getterMember instanceof Field ) {
Opcodes.INVOKESTATIC,
Type.getInternalName( foreignPackageMember.getForeignPackageAccessor() ),
"get_" + getterMember.getName(),
Type.getMethodDescriptor( Type.getType( type ), Type.getType( clazz ) ),
Type.getMethodDescriptor(
Type.getType( type ),
Type.getType( underlyingMember.getDeclaringClass() )
),
false
);
}
@ -976,7 +959,11 @@ else if ( setterMember instanceof Field ) {
Opcodes.INVOKESTATIC,
Type.getInternalName( foreignPackageMember.getForeignPackageAccessor() ),
"set_" + setterMember.getName(),
Type.getMethodDescriptor( Type.getType( void.class ), Type.getType( clazz ), Type.getType( type ) ),
Type.getMethodDescriptor(
Type.getType( void.class ),
Type.getType( foreignPackageMember.getMember().getDeclaringClass() ),
Type.getType( type )
),
false
);
}