Mirror of forum.nim-lang.org

10868 :: A custom '=destroy' hook which takes a 'var T' parameter is deprecated

• drkameleon (orginal) [2024-01-12T12:08:47+01:00] view original

  Somewhere in our internal GMP wrapper, we keep getting this warning:

   Warning: A custom '=destroy' hook which takes a 'var T' parameter is deprecated; it should take a 'T' parameter [Deprecated]
  Run

  The code triggering it: https://github.com/arturo-lang/arturo/blob/master/src/helpers/bignums.nim#L95

  And the finalizeInt implementation: https://github.com/arturo-lang/arturo/blob/master/src/extras/gmp.nim#L801-L802

  Any idea what is going on here exactly and/or how we could solve it?

---

• ggibson (orginal) [2024-01-13T00:31:40+01:00] view original

  How are you able to use finalizers? Shouldn't those be replaced with destroy hooks to be Nim 2 compatible? At least that's what I ran into doing a similar update recently.

---

• MauroG (orginal) [2024-05-21T15:17:21+02:00] view original

  Hi,

  I'm updating a package that has

  new(result, freeConnection)
  Run

  and the same warning

  How should I replace the line to no longer have the warning?

  Thanks

---

• MauroG (orginal) [2024-05-27T12:22:51+02:00] view original

  @ggibson

  sorry if I mention you again but I don't understand how I should replace this code:

  proc freeConnection*(con: ODBCConnection) =
    con.disconnect
    if con.envHandle != nil:
      freeEnvHandle(con.envHandle, con.reporting)
  
  Run

  to have destroy hooks.

---