solc-js
solc-js copied to clipboard
Remove low-level interface (breaking)
The main reason the low-level interface exists is to support older compiler, in the case where the "standard json wrapper" is failing. This was a problem until we had testing enabled for multiple versions (#394).
At this point I'd argue that it is more beneficial to fix any potential wrapper issues and just remove the low-level interface.
Decision in call: For now we make the low level stuff use the new interface (wrapper), and complettly remove it in 0.7.0.
Did #410 fix this? Or is there still something to do?
For now we make the low level stuff use the new interface (wrapper), and complettly remove it in 0.7.0.
I think #410 did the first part.
We have this now:
lowlevel: {
compileSingle: compileJSON,
compileMulti: compileJSONMulti,
compileCallback: compileJSONCallback,
compileStandard: compileStandard
},
features: {
legacySingleInput: compileJSON !== null,
multipleInputs: compileJSONMulti !== null || compileStandard !== null,
importCallback: compileJSONCallback !== null || compileStandard !== null,
nativeStandardJSON: compileStandard !== null
},
compile: compileStandardWrapper,
I think it is fine. The goal here was to remove the lowlevel
+ features
section.