groovy/groovy-core — pull request #13 by jwagenleitner (2012-01-14 02:06:22+00:00)
Title: GROOVY-5216: Sql.newInstance(Map) should handle parameter non-destructively

The url guard statement was moved to the top before any driver loading would take place. Otherwise, just making copies of the args (and also the properties - Oracle adds a protocol prop). Added some other tests for the newInstance(Map) method as well.

I know it looks like quite a bit of change and I may not have placed the tests in the desired class. But thought I'd offer the patch in case it may be of use.