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

fix main return bug #6

Merged
merged 4 commits into from Apr 7, 2021
Merged

fix main return bug #6

merged 4 commits into from Apr 7, 2021

Conversation

wu452148993
Copy link
Contributor

@wu452148993 wu452148993 commented Apr 7, 2021

fix two problem.
1.
image
result:
image
'jump 7 always's goal is empty.
2.
image
result:
image
'set @counter @funcTramp_main' funcTramp_main is empty
Solution:
I translate all 'return' in the main function into 'end', and add 'end' at the end of main function by default.
Problem:
Then we need to add end at the 'end' of all test cases,I think it's a bit strange. lol

@codecov
Copy link

codecov bot commented Apr 7, 2021

Codecov Report

Merging #6 (5c6f3cc) into master (465fc39) will increase coverage by 0.08%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master       #6      +/-   ##
==========================================
+ Coverage   89.72%   89.81%   +0.08%     
==========================================
  Files          29       29              
  Lines        2181     2189       +8     
==========================================
+ Hits         1957     1966       +9     
+ Misses        160      158       -2     
- Partials       64       65       +1     
Impacted Files Coverage Δ
transpiler/main.go 91.51% <100.00%> (+0.15%) ⬆️
transpiler/type_native.go 100.00% <100.00%> (ø)
transpiler/type_mlog.go 67.85% <0.00%> (+1.78%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 465fc39...5c6f3cc. Read the comment docs.

@wu452148993 wu452148993 marked this pull request as ready for review April 7, 2021 11:56
@Vilsol Vilsol merged commit 9038703 into Vilsol:master Apr 7, 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.

None yet

2 participants