struts-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mohan Kishore <mohankish...@yahoo.com>
Subject Re: ModuleConfigImpl holds reference to factory
Date Wed, 05 Feb 2003 03:20:44 GMT

This came up once before: http://nagoya.apache.org/bugzilla/show_bug.cgi?id=16704 Had posted
an explanation with patch - hope that helps....
 David Graham <dgraham1980@hotmail.com> wrote:ModuleConfigImpl implements Serializable
but the ModuleConfigFactory 
interface does not. ModuleConfigImpl has a member variable that stores its 
factory but never actually uses that variable. We have 2 options:

1. Remove the factory variable from ModuleConfigImpl and the constructor 
that sets it. (my favorite)

2. Make the factory variable transient and add a setFactory() method 
because there is no way to reset the variable after deserialization.

Comments? I see no problem with number 1 so I'll make the change in a 
couple days if no one objects.

David





_________________________________________________________________
The new MSN 8: smart spam protection and 2 months FREE* 
http://join.msn.com/?page=features/junkmail


---------------------------------------------------------------------
To unsubscribe, e-mail: struts-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: struts-dev-help@jakarta.apache.org


Mohan Kishore
732 Marlin Ave, Apt 4
Foster City, CA 94404
mohankishore@yahoo.com


---------------------------------
Do you Yahoo!?
Yahoo! Mail Plus - Powerful. Affordable. Sign up now
Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message