Silent error when (re)defining result

[ReneSac]

One of my first errors when I started using nimrod was redefining result. At first I gave up and started using "return" as I would do in any language, but latter I went back and discovered what I was doing wrong. The main problem is that this fails silently, causing the function to return a non-expected result:

proc ret2(): int =
  result = 2

proc ret2NOT(): int =
  var result = 2

echo ret2()     # Prints 2
echo ret2NOT()  # Prints 0

EDIT:

I discovered the real problem:
Renaming "var result" to "var foo" I saw that that if you forget to return a value from a proc, it will return a value anyway because "result" was implicitly initialized. This hides programmer errors. I see no situation where this behavior would be useful, especially because if you just write "proc ret2NOT(): int" the compiler will complain about missing implementation.

[Varriount]

It's arguable whether this is a bug, or a feature. On one hand, it allows those who prefer to use 'return' statements over the implicit result variable the opportunity to use 'result' as a variable with little consequence. On the other hand it does present some confusion for those used to reading code with the implicit result technique.

We could either prevent use of 'result' as a variable, keep the current behavior, or allow a compiler setting which throws a warning, error, or is silent when 'result' is used as a explicit variable. Personally, I'm in favor of the compiler option route, as it gives the most flexibility, but I don't know how much work would be needed to allow the compiler to make such a change.

Additionally, I feel that this behavior should be explicitly mentioned in both the manual and tutorial.

[ReneSac]

Changed my bug a bit, see the EDIT above.

Regarding the "result" variable, I think it should be reserved to avoid confusion. Or at least the compiler should throw a warning by default, because that is bad style. People will still have the shorter "ret" (that I kinda wished nimrod had chosen over result).

[Araq]

implemented.