bytecodealliance / bytecodealliance/wit-bindgen

c#: investigate if BitConverter is needed in cases where we receive a pointer

Open
#1,164 0 comments 0 reactions 0 assignees View on GitHub
gen-c#
Dominant language
Rust
Stars
1.5k
Forks
286
Avg merge
6h 32m
Merged PRs (30d)
19

Description

> Final nit, why we are doing the BitConverter.ToInt32 dance ?

this is coming from one of the other instructions that are generated via the wit-bindgen abi (https://github.com/bytecodealliance/wit-bindgen/blob/5d8fa6196b6382a90592c42aa63ad73f17760797/crates/csharp/src/function.rs#L400-L407)

Maybe we don't need `BitConverter.ToInt32` in other cases? Maybe that something else to clean up (in a separate pr)

_Originally posted by @jsturtevant in https://github.com/bytecodealliance/wit-bindgen/issues/1145#issuecomment-2643626588_

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.