Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.
/ corefx Public archive

Add runtime directive for DbDataReader.GetFieldValue<T> #3917

Merged
merged 1 commit into from
Oct 20, 2015
Merged

Add runtime directive for DbDataReader.GetFieldValue<T> #3917

merged 1 commit into from
Oct 20, 2015

Conversation

natemcmaster
Copy link

Enables using this generic method on UWP apps where in some situations the netnative compiler leaves out the metadata.

@stephentoub
Copy link
Member

cc: @saurabh500

@saurabh500
Copy link
Contributor

@natemcmaster Can you provide more details about the change? A link to the issue ?

@saurabh500
Copy link
Contributor

cc @YoungGah

@natemcmaster
Copy link
Author

When compiled through .NET Native, the metadata for this assembly is not included by default. This xml adds directives to ensure metadata is explicitly included. With out this, calls to GetFieldValue() may result in MissingMetadataException.

cref dotnet/efcore#623

saurabh500 added a commit that referenced this pull request Oct 20, 2015
Add runtime directive for DbDataReader.GetFieldValue<T>
@saurabh500 saurabh500 merged commit 89bb636 into dotnet:master Oct 20, 2015
@saurabh500
Copy link
Contributor

LGTM. I have merged the PR.

@natemcmaster
Copy link
Author

@saurabh500 thanks!

@natemcmaster natemcmaster deleted the getfieldvalue branch January 12, 2016 19:19
@karelz karelz modified the milestone: 1.0.0-rtm Dec 3, 2016
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
Add runtime directive for DbDataReader.GetFieldValue<T>

Commit migrated from dotnet/corefx@89bb636
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants