Clean up commentary.
@@ -320,16 +320,9 @@ type plainMap__ValueAssemblerMap struct {
p *plainMap__Assembler // pointer back to parent, for final insert and state bump
}
-// just embedding plainMap__Assembler works, but generates a lot more and less-inlined assembly than you'd think.
-// explicitly writing the handful of methods we need to have delegated here removes many autogen'd methods from the assembly,
-// and (TODO TEST THIS) also creates much more inlined results for each of these functions.
-
-// oh lol. no actually, on that last part: these methods were already the ones you could just jump for.
-// why? i dunno.
-// this format does do actual inlining instead of a JMP at the end, but...
-// i guess that's probably same-speed and is assembly-longer?
-// i think the asm size overall is small since such fewer methods, but it's odd we lose the JMP style.
-// but whatever, i'm not really optimizing for assembly size. uh, much.
+// we briefly state only the methods we need to delegate here.
+// just embedding plainMap__Assembler also behaves correctly,
+// but causes a lot of unnecessary autogenerated functions in the final binary.
func (ma *plainMap__ValueAssemblerMap) AssembleDirectly(k string) (ipld.NodeAssembler, error) {
return ma.ca.AssembleDirectly(k)
