Skip to content

[wasm] Correctly escape library names when generating symbols for .c #79007

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

Merged
radical merged 8 commits into from
Nov 30, 2022
Merged

[wasm] Correctly escape library names when generating symbols for .c #79007

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
976abf0
[wasm] Correctly escape library names when generating symbols for .c
radical Nov 29, 2022
218c890
[wasm] PInvokeTableGenerator: Use FixupSymbolName in more places
radical Nov 29, 2022
985f6d5
[wasm] *TableGenerator task: Cache the symbol name fixups
radical Nov 29, 2022
7e6718c
[wasm] IcallTableGenerator: fixup assembly name also, when used as a …
radical Nov 29, 2022
a35bce8
cleanup
radical Nov 29, 2022
edec2a9
Merge remote-tracking branch 'origin/main' into pinvoke-escape-name
radical Nov 30, 2022
8ea2884
Add tests
radical Nov 30, 2022
afb6ef3
rename test
radical Nov 30, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
rename test
  • Loading branch information
@radical
radical committed Nov 30, 2022
commit afb6ef351ee9d21da811c613a0dd180236db8133
2 changes: 1 addition & 1 deletion src/mono/wasm/Wasm.Build.Tests/PInvokeTableGeneratorTests.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -560,7 +560,7 @@ public void BuildNativeInNonEnglishCulture(BuildArgs buildArgs, string culture,
"with🚀unicode#"
} })]

public void BuildLibraryWithInvalidCharacters(BuildArgs buildArgs, string[] libraryNames, RunHost host, string id)
public void CallIntoLibrariesWithNonAlphanumericCharactersInTheirNames(BuildArgs buildArgs, string[] libraryNames, RunHost host, string id)
{
buildArgs = ExpandBuildArgs(buildArgs,
extraItems: @$"<NativeFileReference Include=""*.c"" />",
Expand Down