Jacek Sieka

These have different semantics - nim `enum` are in particular a lot more strict than those of C - specially when they have holes or are used as flags, ie...

There are a couple of __attribute__ tags that have corresponding features in Nim - `deprecated`, `alias` etc - it would be nice to have these auto-translated - see also https://github.com/nim-lang/c2nim/pull/252#discussion_r1046775264

Feature addition policy

3

Each addition to the beacon API carries a significant implementation and maintenance cost, since in general, there's an expectation that beacon nodes actually will implement the API as specified. As...

https://github.com/ethereum/consensus-specs/pull/3360 effectively extends the valid lifetime of an attestation/aggregate to 2 epochs - this means that an aggregate that was published at the beginning of a slot now is valid...

chore(nim): task simplification

4

This PR slightly simplifies the way tasks are passed between threads, removing some unnecessary boilerplate while improving type safety by removing casts. This approach works because `nimcall` turns a `proc`...

Use smaller limb size for small stints

5

https://github.com/status-im/nim-stint/blob/master/stint/private/datatypes.nim#L16C8-L16C8 - currently, `Word` is used for all stint sizes, ie `Stint[16]` takes 8 bytes on a 64-bit system. With bit sizes less than the `Word` size, we should be...

fix compile-time hash, port to unittest2

1

* enable compile-time testing for most of the test suite (some parts that were skipping things turned out to be bugs) * port to unittest2 and use its recent compile-time...

```nim let a = -1_000_000.stint(64) let b = 1_000_000.stint(64) check: when sizeof(int) == 8: truncate(a*b, int64) == -1_000_000_000_000'i64 # need 40 bits truncate(b*a, int64) == -1_000_000_000_000'i64 ``` ``` nim c...

https://github.com/status-im/nim-stint/blob/711cda4456c32d3ba3c6c4524135b3453dffeb9c/tests/test_io.nim#L803 When parsing a negative number into a signed stint smaller than the word size, we must left-extend the sign or the general stint representation becomes inconsistent. The given example...

Run nimsuggest with low priority

3

`nimsuggest` and friends are low-priority helpers - as such, it would be preferable to run them with lower scheduling priority so that it doesn't hog valuable CPU when compiling.