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

feat: USING and DROP support #230

Merged
merged 41 commits into from
Feb 24, 2022

Conversation

slavek-kucera
Copy link
Contributor

  • Implements USING and DROP

  • PUSH and POP USING

  • Ordinary, labeled, dependent USINGs

  • Integration with checker in a separate PR

@slavek-kucera slavek-kucera marked this pull request as ready for review February 23, 2022 09:23
parser_library/src/context/using.h Outdated Show resolved Hide resolved
parser_library/src/tagged_index.h Outdated Show resolved Hide resolved
parser_library/src/tagged_index.h Show resolved Hide resolved
parser_library/src/context/using.cpp Outdated Show resolved Hide resolved
parser_library/src/context/using.cpp Show resolved Hide resolved
parser_library/src/context/using.cpp Show resolved Hide resolved
parser_library/src/context/using.cpp Show resolved Hide resolved
Comment on lines +529 to +531
result.reg_offset -= min_disp;
else /* if (result.reg_offset > max_disp) */
result.reg_offset -= max_disp;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure what these two assignments should represent so my question might be irrelevant.

Nevertheless, it seems that when the result.reg_offset < min_disp the min_disp is subtracted from result.reg_offset. Shouldn't it be the other way around? Isn't the end goal to have result.reg_offset in the range <min_disp, max_disp>?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've been considering adding a union with something like out_of_range_by... because that's what it means when the mapping register set consists of invalid registers.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like this idea! Maybe it can be a feature for one of the next PRs

@sonarqubecloud
Copy link

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 78 Code Smells

92.3% 92.3% Coverage
0.0% 0.0% Duplication

@slavek-kucera slavek-kucera merged commit f9dfd05 into eclipse-che4z:development Feb 24, 2022
@slavek-kucera slavek-kucera deleted the feat_using branch February 24, 2022 15:04
@zimlu02 zimlu02 added this to the 1.1.0 milestone Mar 24, 2022
@github-actions
Copy link

🎉 This PR is included in version 1.1.0-beta.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@github-actions
Copy link

🎉 This PR is included in version 1.1.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants