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

[RFC FS-1109] Additional intrinsics for the NativePtr module #11682

Merged
merged 19 commits into from
Jul 13, 2021

Conversation

Happypig375
Copy link
Member

@Happypig375 Happypig375 commented Jun 16, 2021

@dsyme
Copy link
Contributor

dsyme commented Jul 6, 2021

@Happypig375 Where is this at w.r.t. completness, and the RFC?

@Happypig375
Copy link
Member Author

Just missing the RFC, writing now.

@Happypig375
Copy link
Member Author

fsharp/fslang-design#604

@vzarytovskii vzarytovskii requested review from dsyme and TIHan July 12, 2021 07:56
@dsyme
Copy link
Contributor

dsyme commented Jul 13, 2021

@KevinRansom This looks good to me. Do we want "Experimental this is preview" attributes on the additions to FSharp.Core? Thanks

@dsyme
Copy link
Contributor

dsyme commented Jul 13, 2021

@vzarytovskii Approved subject to @KevinRansom 's opinion on whether we need the "this is in preview" library attributes.

@KevinRansom
Copy link
Member

No I think the preview attributes are not necessary. The APIs are not likely to change, they don't support a specific language feature and Dev 17.0 is the target for main anyway.

Copy link
Member

@KevinRansom KevinRansom left a comment

Choose a reason for hiding this comment

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

This looks great

@KevinRansom KevinRansom merged commit 6dd7131 into dotnet:main Jul 13, 2021
@dsyme
Copy link
Contributor

dsyme commented Jul 13, 2021

Cool thank you! So we have our first non-preview F# 5.1 (or F# 6.0 or whatever) feature :)

@dsyme
Copy link
Contributor

dsyme commented Jul 13, 2021

Thank you @Happypig375 :)

@KevinRansom
Copy link
Member

5.1 I think.

@dsyme dsyme changed the title Additional intrinsics for the NativePtr module [RFC FS-1109] Additional intrinsics for the NativePtr module Aug 9, 2021
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