gonum / gonum/hdf5

hdf5: fix []C.hid_t <-> []uint type prunning

Open
#49 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
135
Forks
34
PR merge metrics
No merged PRs in 30d

Description

there are many places where we do:

```go
func f(vs []uint) {
cvs := (*C.hid_t)(unsafe.Pointer(&vs[0]))
C.cfunc(C.int(len(vs)), cvs)
}
```

(or equivalent.)

either replace all these type prunning snippets or insert a `func init()` that checks this type prunning is actually working for the platform/OS/compiler setup at hand.

Contributor guide

No contributing guide indexed for this repository

Research direction

Search the repository for []uint-to-C.hid_t conversions and equivalent cgo calls. Check how these conversions behave across the relevant platform, OS, and compiler setups, then determine whether all call sites need replacement or whether startup validation is sufficient; done means every affected conversion is covered.

Written by the indexing model from the issue text.

Assessment

Tech stack
c, go
Domain
data
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.