Optimise code size of addFunction. NFC

[RReverser]

I think there is more that could be done here - in particular, setWasmTableEntry is only used from libaddfunction and likely could be merged into it to unify data structures and updates - but for now just tackled some low-hanging fruits.

[RReverser]

Huh. Looks like in core2 Closure mangles

$wasmTypeCodes: {
  'i': 0x7f, // i32
#if MEMORY64
  'p': 0x7e, // i64
#else
  'p': 0x7f, // i32
#endif
  'j': 0x7e, // i64
  'f': 0x7d, // f32
  'd': 0x7c, // f64
  'e': 0x6f, // externref
},

into

wb = {qg: 127, p: 127, rg: 126, f: 125, d: 124, e: 111},

I thought quoting was supposed to prevent that...

[RReverser]

Ahh it's because of how jsify generates the variables. It generates them as unquoted when using object literal...

[sbc100]

Ahh it's because of how jsify generates the variables. It generates them as unquoted when using object literal...

Yes, sadly I think the round-tripping that happens when the JS library is processed will remove the quotes.

You can either:
(1) move the object declaration back inside the functions body;
(2) Put the whole thing in a multi-line string.