Random observation...I'm usually not a fan of inner classes but DataMapperLocalSessionScope was an inner class of DataMapper because it only had meaning for DataMapper; it existed to reduce code inside DataMapper.cs based on the current implementation of DataMapper. If its only going to be used in conjunction with DataMapper shouldn't it just be a protected inner class (kind of like log4net.Appender.FileAppender+ExclusiveLock)?
Wouldn't Apache.Ibatis.DataMapper.Session.LocalSessionScope be a better place for it? ----- Original Message ---- From: "[EMAIL PROTECTED]" <[EMAIL PROTECTED]> To: [EMAIL PROTECTED] Sent: Sunday, June 8, 2008 2:20:46 PM Subject: svn commit: r664536 [1/4] - in /ibatis/trunk/cs/V3/src: Apache.Ibatis.DataMapper.SqlClient.Test.2005/ Apache.Ibatis.DataMapper.SqlClient.Test.2005/Fixtures/ Apache.Ibatis.DataMapper.SqlClient.Test.2005/Fixtures/Mapping/ Apache.Ibatis.DataMapper.SqlClie... Author: gbayon Date: Sun Jun 8 11:20:44 2008 New Revision: 664536 URL: http://svn.apache.org/viewvc?rev=664536&view=rev Log: IBATISNET-271 add event support Added: ibatis/trunk/cs/V3/src/Apache.Ibatis.DataMapper.SqlClient.Test.2005/Fixtures/Mapping/EventTest.cs (with props) ibatis/trunk/cs/V3/src/Apache.Ibatis.DataMapper.SqlClient.Test.2005/Fixtures/Modules/EventModule.cs (with props) ibatis/trunk/cs/V3/src/Apache.Ibatis.DataMapper.SqlClient.Test.2005/Fixtures/ScriptBase.cs (with props) ibatis/trunk/cs/V3/src/Apache.Ibatis.DataMapper.SqlClient.Test.2005/Maps/Event.xml ibatis/trunk/cs/V3/src/Apache.Ibatis.DataMapper.SqlClient.Test.2005/bin/Debug/SqlMap.event.config ibatis/trunk/cs/V3/src/Apache.Ibatis.DataMapper/DataMapperLocalSessionScope.cs (with props) [snip]
