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.

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

Jump to bottom

It should be possible to get MetadataImportOptions.All via public API #6748

Closed
nguerrera opened this issue Nov 13, 2015 · 3 comments
Closed

It should be possible to get MetadataImportOptions.All via public API #6748

nguerrera opened this issue Nov 13, 2015 · 3 comments
Labels
Area-Compilers Concept-API This issue involves adding, removing, clarification, or modification of an API. Feature Request help wanted The issue is "up for grabs" - add a comment if you are interested in working on it Interactive-ScriptingLogic Resolution-Fixed The bug has been fixed and/or the requested behavior has been implemented
Milestone
Backlog

Comments

@nguerrera
Copy link
Contributor

Currently, if you get symbols from metadata, the private members are not imported. Internally, there are MetadataImportOptions.All, but there is no way to specify those semantics via public API.

We are investigating using Roslyn to walk metadata for binary analysis scenarios and would need to be able to see the private members too.

cc @tmat
@tmat tmat self-assigned this Nov 13, 2015
@tmat tmat added this to the 1.2 milestone Nov 13, 2015
@tmat
Copy link
Member

tmat commented Nov 13, 2015

The same API would be used by scripting (#r!).

@bkoelman
Copy link
Contributor

Duplicate of #5049?

@jaredpar jaredpar modified the milestones: 2.0, 1.2 Nov 30, 2015
@gafter gafter modified the milestones: 2.0 (Preview 1), 2.0 (RC) Jun 20, 2016
@tmat tmat mentioned this issue Jul 12, 2016
Closed
@jaredpar jaredpar modified the milestones: 2.1, 2.0 (RC) Jul 19, 2016
@jaredpar
Copy link
Member

Pushing out after discussing with @nguerrera

@nguerrera nguerrera mentioned this issue Aug 10, 2016
Open
@jaredpar jaredpar added the help wanted The issue is "up for grabs" - add a comment if you are interested in working on it label Feb 15, 2017
@jaredpar jaredpar modified the milestones: Unknown, 2.1 Feb 15, 2017
@saquibmian saquibmian mentioned this issue Jun 21, 2017
Open
@tmat tmat mentioned this issue Jan 24, 2018
Closed
@tmat tmat removed their assignment Jan 24, 2018
@AlekseyTs AlekseyTs mentioned this issue Jan 26, 2018
Merged
@jcouv jcouv added the Concept-API This issue involves adding, removing, clarification, or modification of an API. label Jan 26, 2018
@AlekseyTs AlekseyTs added the Resolution-Fixed The bug has been fixed and/or the requested behavior has been implemented label Jan 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Area-Compilers Concept-API This issue involves adding, removing, clarification, or modification of an API. Feature Request help wanted The issue is "up for grabs" - add a comment if you are interested in working on it Interactive-ScriptingLogic Resolution-Fixed The bug has been fixed and/or the requested behavior has been implemented
Projects
None yet
Milestone
Backlog
Development

No branches or pull requests
7 participants
@tmat @nguerrera @jaredpar @gafter @bkoelman @AlekseyTs @jcouv