Skip to content

Commit

Permalink
The byteCodePreprocessors doesn't need to be concurrent list
Browse files Browse the repository at this point in the history
- It is appended in the NEW phase, and used in RUNNING phase.

Signed-off-by: David Matějček <david.matejcek@omnifish.ee>
  • Loading branch information
dmatej committed Jan 15, 2023
1 parent eba9b37 commit 2f97375
Showing 1 changed file with 2 additions and 1 deletion.
Expand Up @@ -262,7 +262,7 @@ public final class WebappClassLoader extends GlassfishUrlClassLoader
private boolean hasExternalRepositories;

/** List of byte code pre-processors per webapp class loader. */
private final ConcurrentLinkedQueue<BytecodePreprocessor> byteCodePreprocessors = new ConcurrentLinkedQueue<>();
private List<BytecodePreprocessor> byteCodePreprocessors = new ArrayList<>();

/** myfaces-api uses jakarta.faces packages */
private boolean useMyFaces;
Expand Down Expand Up @@ -579,6 +579,7 @@ public void addTransformer(final ClassFileTransformer transformer) {
public void start() {
LOG.log(DEBUG, "start()");
checkStatus(LifeCycleStatus.NEW);
byteCodePreprocessors = Collections.unmodifiableList(byteCodePreprocessors);
status = LifeCycleStatus.RUNNING;
}

Expand Down

0 comments on commit 2f97375

Please sign in to comment.