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

Use golang.org/x/sys/windows.GetConsoleScreenBufferInfo #115

Merged
merged 1 commit into from
May 16, 2020

Conversation

thaJeztah
Copy link
Contributor

Looks like this is now provided by golang.org/x/sys as well. The other ones not yet (afaics)

Perhaps @tklauser knows if that's within scope of golang.org/x/sys, or if we need to continue implementing it here

(note; I don't have a Windows machine running, so have not tested it)

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah
Copy link
Contributor Author

/cc @dnephin

@dnephin
Copy link
Member

dnephin commented May 16, 2020

Thanks!

@dnephin dnephin merged commit bbd252b into gotestyourself:master May 16, 2020
@thaJeztah thaJeztah deleted the replace_more_syscall branch May 16, 2020 19:21
@tklauser
Copy link

Looks like this is now provided by golang.org/x/sys as well. The other ones not yet (afaics)

Perhaps @tklauser knows if that's within scope of golang.org/x/sys, or if we need to continue implementing it here

I think it would be perfectly acceptable to add the other functions to x/sys/windows as well. Please feel free to send a PR/CL if needed.

gopherbot pushed a commit to golang/sys that referenced this pull request Oct 7, 2020
Relates to gotestyourself/gotestsum#115 (comment)

For details see: https://docs.microsoft.com/en-us/windows/console/SetConsoleCursorPosition

Change-Id: I512428b64af253916c81187cb644e40a6ba418b5
GitHub-Last-Rev: c6c7ba0
GitHub-Pull-Request: #87
Reviewed-on: https://go-review.googlesource.com/c/sys/+/258038
Reviewed-by: Alex Brainman <alex.brainman@gmail.com>
Trust: Alex Brainman <alex.brainman@gmail.com>
Trust: Tobias Klauser <tobias.klauser@gmail.com>
Trust: Ian Lance Taylor <iant@golang.org>
Run-TryBot: Alex Brainman <alex.brainman@gmail.com>
TryBot-Result: Go Bot <gobot@golang.org>
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

Successfully merging this pull request may close these issues.

None yet

3 participants