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 correct call convention for AssocQueryStringW() #63

Merged
merged 1 commit into from
Jan 25, 2023

Conversation

alula
Copy link
Contributor

@alula alula commented Jan 25, 2023

Fixes a minor issue introduced by #62 that broke build on 32-bit Windows targets.

@amodm amodm merged commit 0c59b87 into amodm:main Jan 25, 2023
@amodm
Copy link
Owner

amodm commented Jan 25, 2023

Thanks for the prompt fix for this issue, @alula. This is now out as v0.8.6.

Kryptos-FR added a commit to fragcolor-xyz/shards that referenced this pull request Feb 1, 2023
There is an issue at link time with symbol `AssocQueryStringW` for Windows targets.
It breaks from version 0.8.4 onwards.
PR amodm/webbrowser-rs#63 doesn't seem to fix it.
sinkingsugar added a commit to fragcolor-xyz/shards that referenced this pull request Feb 2, 2023
* fix some issues in wires.cpp

* bonus bump rust crates

* Force use of webbrowser version 0.8.3

There is an issue at link time with symbol `AssocQueryStringW` for Windows targets.
It breaks from version 0.8.4 onwards.
PR amodm/webbrowser-rs#63 doesn't seem to fix it.

* remove webbrowser

---------

Co-authored-by: Nicolas Musset <musset.nicolas@gmail.com>
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.

2 participants