results.Opt as object fields could generate codegen issues

Question

results.Opt as object fields could generate codegen issues

Closed this issue 2 years ago · 3 comments

All Nim versions are affected
#141

import stew/results

type
  ValidatorSig = object
    blob: array[96, byte]
  SyncDutyAndProof = object
    slotSig: Opt[ValidatorSig]
const
  DefaultSyncDutyAndProof =
    SyncDutyAndProof(slotSig: Opt.none(ValidatorSig))

echo DefaultSyncDutyAndProof

/home/runner/.cache/nim/all_tests_d/@mtest_results.nim.c:974:143: error: ‘union <anonymous>’ has no member named ‘_o_1’; did you mean ‘_o_2’?
 N_LIB_PRIVATE NIM_CONST tyObject_SyncDutyAndProof__9atwCZvu9aDnhulGNwFXJ9cFg DefaultSyncDutyAndProof__qmlenLhfD5mHF063hF585Q = {{NIM_FALSE, {._o_1 = {}}}
                                                                                                                                               ^~~~
                                                                                                                                               _o_2

Answer 1 · 2022-10-29T16:38:40.000Z

Seems like a bug in Nim itself

Answer 2 · 2022-11-09T17:37:28.000Z

nim-lang/Nim#20790 should fix it.

Answer 3 · 2023-01-19T12:23:18.000Z

This is fixed both by #150 and upstream fixes