[birt-dev]CheckIn: syncronized problem in plugin loader

["Wei Yan" <wyan@xxxxxxxxxxx>]

Title: [birt-dev]CheckIn: syncronized problem in plugin loader

Summary 

Bugzilla Bug (s) Resolved:

Description:
We use synchronized method to create a class loader which contains a error. The old source code looks like:
If (classLoader != null)
        return classLoader
Synchronized(this)
        classLoader = new ClassLoader()
Return classLoader;

Here, if there mutiple threads wait for the syncrhonize(this), each thread will get different classLoader. Change the source code to:

If (classLoader != null)
        return classLoader;
Synchrnized (this)
        if (classLoader == null)
                classLoader = new ClassLoader
Return classLoader
 
Test Description:
Unit Test (Done by DTE)
 
Files Edited:
        cvs ci -m "synchronized problem" -l "/org.eclipse.birt.core/src/org/eclipse/birt/core/framework/server/Plugin.java"

Files Added:

Notes to Build Team:
 
Notes to Developers:

Notes to QA:
 
Quotes to Documentation:

-Wei Yan