From 7c9dda586e57de1308f07dbaf600be198ce5f901 Mon Sep 17 00:00:00 2001 From: Rob Bygrave Date: Fri, 14 Oct 2016 23:49:00 +1300 Subject: [PATCH] No effective change - format only --- .../server/core/bootup/BootupClasses.java | 85 ++++++++---------- .../ChainedBeanPostConstructListener.java | 90 +++++++++---------- 2 files changed, 82 insertions(+), 93 deletions(-) diff --git a/src/main/java/com/avaje/ebeaninternal/server/core/bootup/BootupClasses.java b/src/main/java/com/avaje/ebeaninternal/server/core/bootup/BootupClasses.java index 1722d0962..3df277068 100644 --- a/src/main/java/com/avaje/ebeaninternal/server/core/bootup/BootupClasses.java +++ b/src/main/java/com/avaje/ebeaninternal/server/core/bootup/BootupClasses.java @@ -42,43 +42,32 @@ public class BootupClasses implements ClassFilter { private final List> entityList = new ArrayList>(); - private final List>> scalarTypeList - = new ArrayList>>(); + private final List>> scalarTypeList = new ArrayList>>(); - private final List>> scalarConverterList - = new ArrayList>>(); + private final List>> scalarConverterList = new ArrayList>>(); + + private final List>> compoundTypeList = new ArrayList>>(); - private final List>> compoundTypeList - = new ArrayList>>(); - // The following objects are instantiiated on first request // there is always a candidate list, that holds the class and an // instance list, that holds the instance. Once a class is instantiiated // (or added) it will get removed from the candidate list - private final List> idGeneratorCandidates - = new ArrayList>(); - - private final List> beanPersistControllerCandidates - = new ArrayList>(); - - private final List> beanPostLoadCandidates - = new ArrayList>(); + private final List> idGeneratorCandidates = new ArrayList>(); - private final List> beanPostConstructListenerCandidates - = new ArrayList>(); - - private final List> beanFindControllerCandidates - = new ArrayList>(); - - private final List> beanPersistListenerCandidates - = new ArrayList>(); - - private final List> beanQueryAdapterCandidates - = new ArrayList>(); - - private final List> serverConfigStartupCandidates - = new ArrayList>(); + private final List> beanPersistControllerCandidates = new ArrayList>(); + + private final List> beanPostLoadCandidates = new ArrayList>(); + + private final List> beanPostConstructListenerCandidates = new ArrayList>(); + + private final List> beanFindControllerCandidates = new ArrayList>(); + + private final List> beanPersistListenerCandidates = new ArrayList>(); + + private final List> beanQueryAdapterCandidates = new ArrayList>(); + + private final List> serverConfigStartupCandidates = new ArrayList>(); private final List idGeneratorInstances = new ArrayList(); private final List beanPersistControllerInstances = new ArrayList(); @@ -88,7 +77,7 @@ public class BootupClasses implements ClassFilter { private final List beanPersistListenerInstances = new ArrayList(); private final List beanQueryAdapterInstances = new ArrayList(); private final List serverConfigStartupInstances = new ArrayList(); - + // single objects private Class changeLogPrepareClass; private Class changeLogListenerClass; @@ -148,7 +137,7 @@ public class BootupClasses implements ClassFilter { // don't automatically instantiate candidates.remove(obj.getClass()); } - } + } } /** @@ -171,13 +160,14 @@ public class BootupClasses implements ClassFilter { public void addPostLoaders(List postLoaders) { add(postLoaders, beanPostLoadInstances, beanPostLoadCandidates); } + /** * Add BeanPostConstructListener instances. */ public void addPostConstructListeners(List postConstructListener) { add(postConstructListener, beanPostConstructListenerInstances, beanPostConstructListenerCandidates); } - + /** * Add BeanFindController instances. */ @@ -192,7 +182,7 @@ public class BootupClasses implements ClassFilter { public void addQueryAdapters(List queryAdapters) { add(queryAdapters, beanQueryAdapterInstances, beanQueryAdapterCandidates); } - + public void addServerConfigStartup(List startupInstances) { add(startupInstances, serverConfigStartupInstances, serverConfigStartupCandidates); } @@ -260,14 +250,14 @@ public class BootupClasses implements ClassFilter { * instantiated only once */ private List createAdd(List instances, List> candidates) { - for (Class cls : candidates) { + for (Class cls : candidates) { T newInstance = create(cls, true); if (newInstance != null) { instances.add(newInstance); } - } - candidates.clear(); // important, clear class list! - return instances; + } + candidates.clear(); // important, clear class list! + return instances; } public ChangeLogPrepare getChangeLogPrepare() { @@ -289,12 +279,11 @@ public class BootupClasses implements ClassFilter { public ReadAuditLogger getReadAuditLogger() { return readAuditLogger; } - - + public List getIdGenerators() { return createAdd(idGeneratorInstances, idGeneratorCandidates); } - + public List getBeanPersistControllers() { return createAdd(beanPersistControllerInstances, beanPersistControllerCandidates); } @@ -302,7 +291,7 @@ public class BootupClasses implements ClassFilter { public List getBeanPostLoaders() { return createAdd(beanPostLoadInstances, beanPostLoadCandidates); } - + public List getBeanPostConstructoListeners() { return createAdd(beanPostConstructListenerInstances, beanPostConstructListenerCandidates); } @@ -310,15 +299,15 @@ public class BootupClasses implements ClassFilter { public List getBeanFindControllers() { return createAdd(beanFindControllerInstances, beanFindControllerCandidates); } - + public List getBeanPersistListeners() { - return createAdd(beanPersistListenerInstances, beanPersistListenerCandidates); + return createAdd(beanPersistListenerInstances, beanPersistListenerCandidates); } - + public List getBeanQueryAdapters() { return createAdd(beanQueryAdapterInstances, beanQueryAdapterCandidates); } - + /** * Return the list of Embeddable classes. */ @@ -388,8 +377,8 @@ public class BootupClasses implements ClassFilter { // Types if (ScalarType.class.isAssignableFrom(cls)) { - scalarTypeList.add((Class>) cls); - interesting = true; + scalarTypeList.add((Class>) cls); + interesting = true; } if (ScalarTypeConverter.class.isAssignableFrom(cls)) { @@ -426,7 +415,7 @@ public class BootupClasses implements ClassFilter { if (BeanFindController.class.isAssignableFrom(cls)) { beanFindControllerCandidates.add((Class) cls); interesting = true; - } + } if (BeanPersistListener.class.isAssignableFrom(cls)) { beanPersistListenerCandidates.add((Class) cls); diff --git a/src/main/java/com/avaje/ebeaninternal/server/deploy/ChainedBeanPostConstructListener.java b/src/main/java/com/avaje/ebeaninternal/server/deploy/ChainedBeanPostConstructListener.java index a4f754300..4ec842e07 100644 --- a/src/main/java/com/avaje/ebeaninternal/server/deploy/ChainedBeanPostConstructListener.java +++ b/src/main/java/com/avaje/ebeaninternal/server/deploy/ChainedBeanPostConstructListener.java @@ -10,47 +10,47 @@ import java.util.List; */ public class ChainedBeanPostConstructListener implements BeanPostConstructListener { - private final List list; + private final List list; - private final BeanPostConstructListener[] chain; + private final BeanPostConstructListener[] chain; - /** - * Construct given the list of BeanPostCreate's. - */ - public ChainedBeanPostConstructListener(List list) { - this.list = list; - this.chain = list.toArray(new BeanPostConstructListener[list.size()]); - } - - /** - * Register a new BeanPostCreate and return the resulting chain. - */ - public ChainedBeanPostConstructListener register(BeanPostConstructListener c) { - if (list.contains(c)){ - return this; - } else { - List newList = new ArrayList(); - newList.addAll(list); - newList.add(c); - - return new ChainedBeanPostConstructListener(newList); - } - } - - /** - * De-register a BeanPostCreate and return the resulting chain. - */ - public BeanPostConstructListener deregister(BeanPostConstructListener c) { - if (!list.contains(c)){ - return this; - } else { - ArrayList newList = new ArrayList(); - newList.addAll(list); - newList.remove(c); - - return new ChainedBeanPostConstructListener(newList); - } - } + /** + * Construct given the list of BeanPostCreate's. + */ + public ChainedBeanPostConstructListener(List list) { + this.list = list; + this.chain = list.toArray(new BeanPostConstructListener[list.size()]); + } + + /** + * Register a new BeanPostCreate and return the resulting chain. + */ + public ChainedBeanPostConstructListener register(BeanPostConstructListener c) { + if (list.contains(c)) { + return this; + } else { + List newList = new ArrayList(); + newList.addAll(list); + newList.add(c); + + return new ChainedBeanPostConstructListener(newList); + } + } + + /** + * De-register a BeanPostCreate and return the resulting chain. + */ + public BeanPostConstructListener deregister(BeanPostConstructListener c) { + if (!list.contains(c)) { + return this; + } else { + ArrayList newList = new ArrayList(); + newList.addAll(list); + newList.remove(c); + + return new ChainedBeanPostConstructListener(newList); + } + } /** * Return the size of the chain. @@ -69,11 +69,11 @@ public class ChainedBeanPostConstructListener implements BeanPostConstructListen * Fire postLoad on all registered BeanPostCreate implementations. */ @Override - public void postConstruct(Object bean) { - for (int i = 0; i < chain.length; i++) { - chain[i].postConstruct(bean); - } - } + public void postConstruct(Object bean) { + for (int i = 0; i < chain.length; i++) { + chain[i].postConstruct(bean); + } + } @Override public void autowire(Object bean) { @@ -81,7 +81,7 @@ public class ChainedBeanPostConstructListener implements BeanPostConstructListen chain[i].autowire(bean); } } - + @Override public void postCreate(Object bean) { for (int i = 0; i < chain.length; i++) {