OK, just evaluated your changes.. 

A) seems to me your jumping through hoops there. wouldn't a simple "layer null 
check" with warning to the log in the unexecute() method have sufficed?

B) you create a new constructor with added layer parameter. this is used only 
by the tools you modded accordingly, others will probably still NPE, even with 
your changes? no? 

C) i agree with the dispose refinements though

..ede

------------------------------------------------------------------------------
Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS,
MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current
with LearnDevNow - 3,200 step-by-step video tutorials by Microsoft
MVPs and experts. SALE $99.99 this month only -- learn more at:
http://p.sf.net/sfu/learnmore_122412
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Reply via email to