Skip to content

feat(windows): add error code from async operation#425

Merged
deadprogram merged 1 commit intotinygo-org:devfrom
acouvreur:add-windows-async-error-details
Apr 10, 2026
Merged

feat(windows): add error code from async operation#425
deadprogram merged 1 commit intotinygo-org:devfrom
acouvreur:add-windows-async-error-details

Conversation

@acouvreur
Copy link
Copy Markdown
Contributor

@acouvreur acouvreur commented Mar 17, 2026

With the addition of IAsyncInfo in winrt-go: saltosystems/winrt-go#118

We can now extract the actual asynchronous operation error code.

Before:

write device name: failed to write device name: async operation failed with status 3                                                                            

After:

write device name: failed to write device name: async operation failed with status 3: HRESULT 0x80070005                                                                                

This bug report is from my actual code, I have yet to investigate why I get a HRESULT 0x80070005 😄

But, I can actually debug it now.

@acouvreur
Copy link
Copy Markdown
Contributor Author

@jagobagascon what do you think about this PR ?

I want to be able to use the newly introduced errors in #421 but I did not implement parsing an error from a code.

And application error range codes are not supported by the current error implementation.

@deadprogram
Copy link
Copy Markdown
Member

@jagobagascon or anyone else have any feedback on this PR?

Copy link
Copy Markdown
Member

@jagobagascon jagobagascon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not much to add. LGTM

@deadprogram
Copy link
Copy Markdown
Member

Thanks for the addition @acouvreur and to @jagobagascon for review. Now merging.

@deadprogram deadprogram merged commit 9eaff1b into tinygo-org:dev Apr 10, 2026
4 checks passed
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.

3 participants