Skip to content

Fix Attribute Serialization - #15152

Merged
KevinRansom merged 6 commits into
dotnet:mainfrom
KevinRansom:fixattributeserialization
May 2, 2023
Merged

KevinRansom merged 6 commits into
dotnet:mainfrom
KevinRansom:fixattributeserialization

Conversation

@KevinRansom

@KevinRansom KevinRansom commented Apr 28, 2023 •

Copy link
Copy Markdown
Contributor

Fixes: #15140

F# Attribute blob serialization for open generics created an invalid blob. This addresses that issue.

The fix is to use typdefof, which serializes correctly for open generics.

Todo:

  • Add regression

@KevinRansom
KevinRansom requested a review from a team as a code owner April 28, 2023 03:01
@KevinRansom
KevinRansom force-pushed the fixattributeserialization branch from b15a2b5 to f99deb3 Compare April 28, 2023 21:01
@KevinRansom
KevinRansom force-pushed the fixattributeserialization branch from f99deb3 to 68372c6 Compare April 29, 2023 00:32
Comment thread tests/FSharp.Compiler.ComponentTests/FSharp.Compiler.ComponentTests.fsproj Outdated
@KevinRansom
KevinRansom enabled auto-merge (squash) May 2, 2023 16:41
@KevinRansom
KevinRansom merged commit 124ca6b into dotnet:main May 2, 2023
@KevinRansom
KevinRansom deleted the fixattributeserialization branch March 21, 2024 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Invalid custom attribute blob on FSharpList..ctor

4 participants