dep(punycode.js), reduced indirection, spread operator

[msimerson]

• punycode.js: also update package.json
• remove unused indirection (init_wildduck_shared)
• use es6 ... to reduce boilerplate
• disable eslint prefer-const rule

[andris9]

I don't understand why did you change init_wildduck_shared and open_database functions? open_database is called by init_wildduck_shared and this is called by plugin.register_hook('init_… hooks.

[louis-lau]

Also, prefer-const seems like a good rule to have?

[msimerson]

Why not call open_database directly in register? What is the logic behind calling a function which does nothing besides calling another function with rearranged arguments.

I also agree with prefer-const, but that doesn't seem to be the preferred style. I've moved the prefer-const conversation to PR #36

[msimerson]

Now I understand your confusion. I forgot to update the call in register to open_database. PR updated.