[VM/lib] Merge constants and builtin's?

[github-actions]

[VM/lib] Merge constants and builtin's?

Do we really - really - need another "constant" type? I doubt it whether it makes any serious performance difference, with the only exception being constants like true, false, etc.

But then, it also over-complicates documentation generation for constants.

So, we should either make documentation possible for constants as well, or merge the two things into one concept

arturo/src/vm/lib.nim

Line 113 in ce734e7

# TODO(VM/lib) Merge constants and builtin's?

            )

# TODO(VM/lib) Merge constants and builtin's?
#  Do we really - really - need another "constant" type? I doubt it whether it makes any serious performance difference, with the only exception being constants like `true`, `false`, etc.
#  But then, it also over-complicates documentation generation for constants.
#  So, we should either make documentation possible for constants as well, or merge the two things into one concept
#  labels: vm, library, enhancement, open discussion
template constant*(n: string, alias: SymbolKind, description: string, v: Value):untyped =
    Syms[n] = (v)
    Syms[n].info = "[" & static (instantiationInfo().filename).replace(".nim") & ":" & $(static (instantiationInfo().line)) & "] " & description
ndex e098b999e..4d261283b 100644
++ b/src/vm/package.nim

07c8d3ba9b413ceb346e0c3ce2d4dcf661f97847

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[stale]

Closing issue as stale.

[drkameleon]

Re-opening as still valid

[stale]

Closing issue as stale.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[stale]

Closing issue as stale.