Display error messages in the wasm demo #31
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
概要
wasm版demoにおいて、フォーマットに失敗した際にエラーメッセージを表示するように変更しました。 (#25 )
ただし、UroboroSQLFmtErrorを単純に文字列に変更して表示しているため、エラー内容はわかりにくいものとなっています。
エラーメッセージの内容については別で議論する必要があると思うので、とりあえずドラフトの形でPRを作成しました。
実装
以前は
format_sql_for_wasm
でフォーマット結果を格納したポインタを返していました。しかし、今回の変更ではフォーマット結果とエラーメッセージの2つの文字列を返したいため、実装を以下のように変更しました。
RESULT
、ERROR_MSG
とする。現状大きさ50000としている。)RESULT
とERROR_MSG
の先頭アドレスを取得 (Rust側でアドレスを教える関数を作成して実現)format_sql_for_wasm
が呼び出されたらフォーマットを実行し、正常ならば結果をRESULT
に、エラーの場合はエラーメッセージをERROR_MSG
に格納RESULT
、ERROR_MSG
から結果とエラーメッセージを取得して表示