Mirror of forum.nim-lang.org

2817 :: high(uint) is invalid

• jordan (orginal) [2017-02-24T15:33:46+01:00] view original

  Is it possible that the latest NIM update from GitHub renders

  high(uint)

  invalid? The error reported here is

  invalid argument for 'high'

  Or did I get something wrong?

---

• Jehan (orginal) [2017-02-24T17:12:15+01:00] view original

  Seems to be a bug with semLowHigh() in compiler/semexprs.nim missing the cases for tyUInt and tyUInt64.

---

• Araq (orginal) [2017-02-24T17:20:43+01:00] view original

  That's not a bug, it's a documented implementation limitation.

---

• jordan (orginal) [2017-02-24T17:22:21+01:00] view original

  I see. Whatever it is it affects table allocation as well:

  newTableuint,string

  produces all sort of noise:

  lib/pure/collections/tables.nim(434, 27) template/generic instantiation from here
  lib/pure/collections/tables.nim(319, 39) template/generic instantiation from here
  lib/pure/collections/tableimpl.nim(98, 21) template/generic instantiation from here
  lib/pure/collections/tableimpl.nim(53, 14) template/generic instantiation from here
  lib/pure/collections/tableimpl.nim(43, 12) Error: type mismatch: got (uint)

  while newTableint,string works fine.

---

• jordan (orginal) [2017-02-24T17:28:19+01:00] view original

  As for the limitation, the systems lib does not say anything about limitations on high() and as far as I remember it worked last week or so.

  Anyway, is there a way to use tables with unsigned data types or do I need to map it to signed ones?

---

• Araq (orginal) [2017-02-24T17:48:36+01:00] view original

  newTableuint, string should work and if not that a bug/regression. That seems to have nothing do to with high(uint) though?

---

• jordan (orginal) [2017-02-24T18:21:55+01:00] view original

  I definitely does not work at the moment. Would you want me to file a bug report?

  It is actually not a big problem as I can always map the uint64 to int for the hash tables. The seq[] sorting seems to work so it is ok for me.

---

• Araq (orginal) [2017-02-24T18:29:04+01:00] view original

  Would you want me to file a bug report?

  Yes please.

---