#1046 ChemObject.clone() throws NPE

[Goglepox]

chemObject.clone() throws NPE when cloning object with null as value in the properties map.
chemObject.setProperty("key", null); // this works because lazyProperties() creates a HashMap also setProperties(Map) allow for any Map implementation
chemObject.clone(); this fails now because clone uses a Hashtable as the Map implementation which dose not allow null as value.

[Goglepox]

Test

[Egon Willighagen]

Arvid, better to not catch the Exception, as it will mask where the NPE occurred. Applied to cdk-1.2.x.

[Egon Willighagen]

Patch that fixes the NPE problem upon clone available at:

https://sourceforge.net/tracker/?func=detail&aid=2977453&group_id=20024&atid=320024

[Egon Willighagen]

Bug fix patch has been applied.