Hi Adam,
I agree, Thanks for the suggestion. I will provide the new patch with the changes.

-Thanks,
Srinath K.

Adam Winer wrote:

I added one comment to the issue tracker:  this patch
adds global synchronization to the application of changes.
This seems higly excessive!  The responsibility for
synchronization should belong on the implementation of
ChangeManager.getComponentChanges(), not on its
use.  If you used a CopyOnWriteArrayList for these
lists, wouldn't that be sufficient?

-- Adam



On 4/26/07, Srinathreddy Komatireddy
<[EMAIL PROTECTED]> wrote:

Hi,

   Can someone please review ADFFACES-466
<https://issues.apache.org/jira/browse/ADFFACES-466>. I have submitted a
patch describing the issues and changes.

-Thanks,
Srinath K



Reply via email to