Skip to content

Remove Windows *A declarations #9054

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

Jump to bottom
Closed
marler8997 opened this issue Jun 9, 2021 · 2 comments
Closed

Remove Windows *A declarations #9054

marler8997 opened this issue Jun 9, 2021 · 2 comments
Labels
breaking Implementing this issue could cause existing code to no longer compile or have different behavior. standard library This issue involves writing Zig code for the standard library.
Milestone
0.9.0

Comments

@marler8997
Copy link
Contributor

Zig has (correctly) moved away from calling the legacy *A versions of functions and moved onto the *W versions (see #534). In light of that, it seems appropriate to remove these *A declarations from std.

The only hesitation would be that if applications actually need these declarations that we make it easy to get them, so maybe we will want to wait for the package manager first?
@SpexGuy SpexGuy added breaking Implementing this issue could cause existing code to no longer compile or have different behavior. standard library This issue involves writing Zig code for the standard library. labels Jun 9, 2021
@Vexu Vexu added this to the 0.9.0 milestone Jun 9, 2021
@0x08088405
Copy link
Contributor

0x08088405 commented Jun 9, 2021 •
edited
Loading

I was actually considering this while writing #9037 and I can personally vouch for this, however there was some disagreement about it already previously and it seems it's been thought about in #4426 and was ruled out. Although it seems like the fate of WINAPI definitions in std might heavily depend on #6600 as well.

@andrewrk
Copy link
Member

I'm in favor of completely removing the A declarations. This issue is a duplicate of #4426

@squeek502 squeek502 mentioned this issue Jan 19, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
breaking Implementing this issue could cause existing code to no longer compile or have different behavior. standard library This issue involves writing Zig code for the standard library.
Projects
None yet
Milestone
0.9.0
Development

No branches or pull requests
5 participants
@andrewrk @marler8997 @SpexGuy @Vexu @0x08088405