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

Stack decouple and compiler unary minus support #68

Merged
merged 6 commits into from
Sep 9, 2023
Merged

Stack decouple and compiler unary minus support #68

merged 6 commits into from
Sep 9, 2023

Conversation

ldilley
Copy link
Member

@ldilley ldilley commented Sep 9, 2023

Proposed Change(s)

  • Remove unnecessary Object * cast in op_asn().
  • Fix intentional switch fallthrough warning in a portable manner using goto.
  • Decouple stack from VM so the data structure can be used elsewhere.
  • Add compiler support for unary minus/negative sign.

@ldilley ldilley added enhancement Enhance an existing feature new feature Request something new labels Sep 9, 2023
@ldilley ldilley added this to the 0.1.0 milestone Sep 9, 2023
@ldilley ldilley requested a review from Bamalot September 9, 2023 18:00
@ldilley ldilley self-assigned this Sep 9, 2023
@AppVeyorBot
Copy link

✅ Build concoct 0.1.145 completed (commit 11104cd4fe by @ldilley)

@AppVeyorBot
Copy link

✅ Build concoct 0.1.45 completed (commit 11104cd4fe by @ldilley)

@AppVeyorBot
Copy link

✅ Build concoct 0.1.146 completed (commit 1a9812a1fc by @ldilley)

@sonarcloud
Copy link

sonarcloud bot commented Sep 9, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@AppVeyorBot
Copy link

✅ Build concoct 0.1.46 completed (commit 1a9812a1fc by @ldilley)

@ldilley ldilley merged commit c4c3554 into Concoctist:master Sep 9, 2023
6 checks passed
@ldilley ldilley mentioned this pull request Oct 1, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Bamalot Bamalot Bamalot approved these changes

Assignees

@ldilley ldilley

@Bamalot Bamalot

Labels
enhancement Enhance an existing feature new feature Request something new
Projects
Concoct
Status: Complete
Milestone
0.1.0
Development

Successfully merging this pull request may close these issues.
3 participants
@ldilley @AppVeyorBot @Bamalot