stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

8298449: Unnecessary Vector usage in MetaData.ProxyPersistenceDelegate

Browse Source 
Reviewed-by: serb
This commit is contained in:
Andrey Turbanov 2023-01-05 16:49:23 +00:00
parent be64bf8cf0
commit b908388e81
1 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/java.desktop/share/classes/java/beans/MetaData.java
View File

@ -1,5 +1,5 @@
/*
* Copyright (c) 2000, 2021, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2000, 2022, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@ -155,9 +155,8 @@ static final class ProxyPersistenceDelegate extends PersistenceDelegate {
// This unappealing hack is not required but makes the
// representation of EventHandlers much more concise.
java.lang.reflect.InvocationHandler ih = java.lang.reflect.Proxy.getInvocationHandler(p);
if (ih instanceof EventHandler) {
EventHandler eh = (EventHandler)ih;
Vector<Object> args = new Vector<>();
if (ih instanceof EventHandler eh) {
ArrayList<Object> args = new ArrayList<>();
args.add(type.getInterfaces()[0]);
args.add(eh.getTarget());
args.add(eh.getAction());
@ -165,7 +164,9 @@ static final class ProxyPersistenceDelegate extends PersistenceDelegate {
args.add(eh.getEventPropertyName());
}
if (eh.getListenerMethodName() != null) {
args.setSize(4);
if (args.size() == 3) {
args.add(null);
}
args.add(eh.getListenerMethodName());
}
return new Expression(oldInstance,