I recently submitted this patch: http://216.121.112.228/browse/NH-2126, which I needed for work on the product my team is building using NH. In getting into the code, it looks like the hbm2ddl code could use a more complete fix to the issue I addressed: IDbCommand objects not being created using the same mechanisms the rest of NH uses (when appropriate).
In my opinion, the way the Execute(...) methods are chained could use some cleaning up as well. I'd like to invest some time in giving some TLC to the hbm2ddl code -- does anyone support this effort, i.e., if I do it, assuming it's quality code, what are the chances that these hbm2ddl cleanups would be considered a worthwhile effort for 3.0? I can give more detail about my planned changes if wanted. Please advise. Thanks Chris
