search

stacks-network / clarity-wasm

`clar2wasm` is a compiler for generating WebAssembly from Clarity.
GNU General Public License v3.0
12 stars 9 forks source link

Compilation errors handling #386

Closed csgui closed 1 month ago

csgui commented 2 months ago

Interpreter errors (i.e: syntax error, etc...) should be mapped to Wasm compilation errors.

smcclellan commented 1 month ago

@csgui to add more info to this.

csgui commented 1 month ago

Following a closer look, compilation errors are being handled as a GeneratorError::InternalError. If a new error type is needed in the future, that can be done later.

Closing this issue.

Acaccia commented 4 weeks ago

@csgui super weird take, why shouldn’t there be clear compilation errors returned to the Clarity dev? Shouldn’t InternalErrors be for the cases where we have a logic error in the compiler code?

csgui commented 4 weeks ago

@Acaccia As you noticed this issue is related to the #421. I can say that it was closed by mistake. Let's use the issue #421 to work on compilation-time errors.