Introduce a Lent[T] shim for the lent type and use it in the results module
#88
Conversation
|
it's a bit early for this - ie it would make a central piece of the code behave in a different way between 1.2 and 1.6 - when we have both versions in unit tests for all relevant repos, we can also start differentiating between the two versions. |
|
We can have this disabled by default when it's time to switch to Nim 1.6. |
the other way around, let's add the flag now, and when we're comfortable, we make it default - the active choice should be to enable it |
* Worked-around a bug where Nim comlains that the return value is not a path expression in functions lacking explicit return statements * Worked-around a limitation that Nim doesn't allow: let foo: lent T = makeT()
| ## Fetch value of result if set, or raise Defect | ||
| ## Exception bridge mode: raise given Exception instead | ||
| ## See also: Option.get | ||
| assertOk(self) | ||
| self.v | ||
| return self.v |
There was a problem hiding this comment.
this looks like something to fix in the compiler rather than here, or we're in for a massive refactoring
in particular, because this change doesn't just impact our code - it impacts all projects using stew - we should not be handing them an unwelcome compiler bug present |
|
Is this still relevant in this repo since |
No description provided.