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

Exported Types needs to be translated from .Net Types to IL Types #2

Closed
pjbgf opened this issue May 25, 2017 · 1 comment
Closed

Exported Types needs to be translated from .Net Types to IL Types #2

pjbgf opened this issue May 25, 2017 · 1 comment
Assignees
Labels
Milestone

Comments

@pjbgf
Copy link
Owner

pjbgf commented May 25, 2017

Current IL

.method private hidebysig System.Void ConsoleApp1.Program::Main(System.String[]) cil managed
{
.entrypoint
// Code size 13
.maxstack 8
IL_0000: nop
IL_0001: ldstr "Hello World!"
IL_0006: call System.Void System.Console::WriteLine(System.String)
IL_000b: nop
IL_000c: ret
}

Expected IL

.method private hidebysig void ConsoleApp1.Program::Main(string[]) cil managed
{
.entrypoint
// Code size 13
.maxstack 8
IL_0000: nop
IL_0001: ldstr "Hello World!"
IL_0006: call void System.Console::WriteLine(string)
IL_000b: nop
IL_000c: ret
}
@pjbgf pjbgf added this to the 1.0 milestone May 25, 2017
@pjbgf pjbgf added the bug label May 25, 2017
pjbgf pushed a commit that referenced this issue May 25, 2017
not full method is for implementations, moved conditional inside Hand…
@pjbgf pjbgf modified the milestones: Future, 1.0 May 28, 2017
@Trazarw
Copy link
Collaborator

Trazarw commented May 29, 2017

#22

@pjbgf pjbgf modified the milestones: v0.7.0, Future May 29, 2017
pjbgf added a commit that referenced this issue May 31, 2017
pjbgf pushed a commit that referenced this issue Jun 2, 2017
* cli type for System.Int32 should be int32

* Covering all base types and spliting formatting and conversion
@pjbgf pjbgf closed this as completed Jun 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants