Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[superseded] ref syntax for lvalue expressions: byRef: myref=x[1].foo[2] #11824

Closed
wants to merge 14 commits into from
Closed

[superseded] ref syntax for lvalue expressions: byRef: myref=x[1].foo[2] #11824

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
848a9ac
nim now has ref-alias and ptr-alias for lvalue expressions
timotheecour Jul 25, 2019
774ff5d
address comments
timotheecour Jul 25, 2019
042fd91
byRef with option to export
timotheecour Jul 29, 2019
cc37767
fixup
timotheecour Jul 29, 2019
cda97f1
fixup
timotheecour Jan 12, 2020
a9cd644
add changelog + since
timotheecour Jan 12, 2020
87d6cf9
removed byPtr
timotheecour Jan 13, 2020
ad8a731
byRef => byAddr
timotheecour Jan 18, 2020
6b48b9d
_
timotheecour Jan 20, 2020
0e66b9a
fix comments
timotheecour Jan 23, 2020
1d6b6bd
address comment
timotheecour Jan 23, 2020
edf6960
disallow export: byAddr: barx*=foo.bar.x
timotheecour Jan 23, 2020
2bec2dc
now using explicit export in test
timotheecour Jan 23, 2020
b1e338f
improve splitDefinition
timotheecour Jan 23, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix comments
  • Loading branch information
@timotheecour
timotheecour committed Jan 23, 2020
commit 0e66b9a8bb71c9a4c9f0ea243f13b6f1c48b78d8
2 changes: 1 addition & 1 deletion changelog.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@

- Added `os.isRelativeTo` to tell whether a path is relative to another

- Added `sugar.byAddr` allowing a ref syntax for lvalue expressions
- Added `sugar.byAddr` allowing a reference syntax for lvalue expressions, analog to C++ `auto& a = expr`.

## Library changes

Expand Down
6 changes: 3 additions & 3 deletions lib/pure/sugar.nim
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -384,9 +384,9 @@ proc splitDefinition*(def: NimNode): tuple[lhs: NimNode, rhs: NimNode, exported:
expectKind(result.lhs, nnkIdent)

macro byAddr*(def: untyped): untyped {.since: (1,1).} =
## Defines a ref alias for lvalue expressions. The expression is evaluated
## only once, and any side effects will only be evaluated once, at declaration
## time.
## Defines a reference syntax for lvalue expressions, analog to C++ `auto& a = expr`.
## The expression is evaluated only once, and any side effects will only be
## evaluated once, at declaration time.
runnableExamples:
var x = @[1,2,3]
byAddr: x1=x[1]
Expand Down