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.

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

seq self-assignment order-of-evaluation broken #9684

Closed
arnetheduck opened this issue Nov 11, 2018 · 1 comment

Comments

Projects
None yet
4 participants
@arnetheduck
Copy link
Contributor

commented Nov 11, 2018

there are two items in the list prior to assignment, not 1

[arnetheduck@tempus tmp]$ more test.nim
var s2 = @[2, 2]

s2 = @[s2.len]

echo s2
[arnetheduck@tempus tmp]$ nim --version
Nim Compiler Version 0.19.0 [Linux: amd64]
Compiled at 2018-09-26
Copyright (c) 2006-2018 by Andreas Rumpf

git hash: f6c5c636bb1a1f4e1301ae0ba5a8afecef439132
active boot switches: -d:release
[arnetheduck@tempus tmp]$ nim c -r test
Hint: used config file '/home/arnetheduck/.choosenim/toolchains/nim-0.19.0/config/nim.cfg' [Conf]
Hint: system [Processing]
Hint: test [Processing]
Hint:  [Link]
Hint: operation successful (12272 lines compiled; 0.111 sec total; 16.395MiB peakmem; Debug Build) [SuccessX]
Hint: /home/arnetheduck/tmp/test  [Exec]
@[1]

@arnetheduck arnetheduck changed the title seq self-assignment broken seq self-assignment order-of-evaluation broken Nov 11, 2018

@Araq Araq added the Showstopper label Nov 15, 2018

Araq added a commit that referenced this issue Nov 16, 2018

@Clyybber Clyybber referenced this issue May 9, 2019

Merged

Fix #9844 #11216

@Clyybber

This comment has been minimized.

Copy link
Contributor

commented May 9, 2019

Fixed by #11216

Clyybber added a commit to Clyybber/Nim that referenced this issue May 9, 2019

Clyybber added a commit to Clyybber/Nim that referenced this issue May 9, 2019

@narimiran narimiran closed this May 10, 2019

Araq added a commit that referenced this issue May 10, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.