Mirror of forum.nim-lang.org

3835 :: Strange regression in symbol binding inside a macro

• xomachine (orginal) [2018-05-24T00:33:39+02:00] view original

  I've found strange bug in the devel branch. Take a look at MWE:

  # moduleA.nim
  from streams import newStringStream, readData, writeData
  from macros import quote
  
  macro bindme*(): untyped =
    quote do:
      var tst = "sometext"
      var ss = newStringStream("anothertext")
      ss.writeData(tst[0].addr, 2)
      discard ss.readData(tst[0].addr, 2) # <= comment this out to make compilation successful
  Run

  # moduleB.nim
  from moduleA import bindme
  
  bindme()
  Run

  The command

   $ nim c moduleB.nim
  Run

  produces the error message: Error: attempting to call undeclared routine: 'readData'.

  If the highlighted line is commented out the error disappears. On the master branch the compilation successful regardless of the line presence.

  Nim version (from devel):

  
  Nim Compiler Version 0.18.1 [Linux: amd64]
  Compiled at 2018-05-17
  Copyright (c) 2006-2018 by Andreas Rumpf
  
  git hash: 072cd073ba40d05ace580292735b5b385544d3e7
  active boot switches: -d:release -d:nativeStackTrace
  Run

  Nim version (from master):

  
  Nim Compiler Version 0.18.0 [Linux: amd64]
  Copyright (c) 2006-2018 by Andreas Rumpf
  
  active boot switches: -d:release -d:nativeStackTrace
  Run

---

• jxy (orginal) [2018-05-24T05:09:40+02:00] view original

  I think I've been seeing this too, and I was walking around it by exporting more symbols. Can you submit a bug report?

---

• Araq (orginal) [2018-05-24T09:30:08+02:00] view original

  Is it this bug? https://github.com/nim-lang/Nim/issues/7829

---

• xomachine (orginal) [2018-05-24T13:39:34+02:00] view original

  It does not look like so. My investigation with git bisect shows that the problem have been introduced with this commit

---

• xomachine (orginal) [2018-05-26T21:05:03+02:00] view original

  The problem did not disapeared after fixing https://github.com/nim-lang/Nim/issues/7829 So it is unrelated. I'll post an issue on github soon.

---