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

eraseLine fails with OverflowError on Windows 10 #7043

Closed
Feinbube opened this issue Jan 8, 2018 · 7 comments
Closed

eraseLine fails with OverflowError on Windows 10 #7043

Feinbube opened this issue Jan 8, 2018 · 7 comments

Comments

@Feinbube
Copy link

Feinbube commented Jan 8, 2018

I encountered this error when I tried to install Nim on Windows 10 using choosenim.

Original Error:

Downloading Nim 0.17.2 from nim-lang.org
Traceback (most recent call last)
choosenim.nim(155)       choosenim
choosenim.nim(136)       performAction
choosenim.nim(74)        choose
choosenim.nim(61)        chooseVersion
choosenim.nim(23)        installVersion
download.nim(227)        download
download.nim(221)        downloadImpl
download.nim(169)        downloadFile
download.nim(154)        downloadFileNim
httpclient.nim(1256)     downloadFile
httpclient.nim(951)      parseBody
httpclient.nim(892)      recvFull
httpclient.nim(866)      reportProgress
download.nim(150)        onProgressChanged
terminal.nim(657)        showBar
terminal.nim(396)        eraseLine
system.nim(2724)         sysFatal
Error: unhandled exception: over- or underflow [OverflowError]

Minimal example to reproduce:

import terminal
eraseLine()

Error:

Traceback (most recent call last)
terminal.nim(657)        test
terminal.nim(396)        eraseLine
system.nim(2724)         sysFatal
Error: unhandled exception: over- or underflow [OverflowError]
@data-man
Copy link
Contributor

data-man commented Jan 8, 2018

@Feinbube
Try to replace ht * wt to wt in the eraseLine proc.

@dom96
Copy link
Contributor

dom96 commented Jan 8, 2018

This is a known issue and has been fixed in devel IIRC

(Will be fixed in next choosenim)

@dom96 dom96 closed this as completed Jan 8, 2018
@data-man
Copy link
Contributor

data-man commented Jan 8, 2018

@dom96
No, is not fixed.

@dom96
Copy link
Contributor

dom96 commented Jan 8, 2018

822da4b

@ghost
Copy link

ghost commented Jan 8, 2018

@data-man ^

@data-man
Copy link
Contributor

data-man commented Jan 8, 2018

Maybe to begin to check code before closing a issue?
Implementation isn't correct!

@data-man
Copy link
Contributor

data-man commented Jan 8, 2018

#7044

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants