Attached is patch number 4.
- Michael Spencer
ms-coff-patch-4.patch (45.2 KB)
Hi Michael,
Looks great to me, with the caveat that I know nothing about COFF.
A few comments:
- The term symbol is already very overloaded, I would avoid adding a
new 'class Symbol'.
- I find it annoying to have more versions of write little endian
32-bit data, but I'm not sure what the right solution is. We can clean
this up later, perhaps.
- There are a couple of instances of: "}\nelse" instead of "} else".
Thanks!
- Daniel
Would COFFSymbol be better? I'll fix the others and commit, thanks.