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

AML: Alias opcode support required #136

Open
rw-vanc opened this issue Jan 26, 2023 · 2 comments
Open

AML: Alias opcode support required #136

rw-vanc opened this issue Jan 26, 2023 · 2 comments

Comments

@rw-vanc
Copy link
Contributor

rw-vanc commented Jan 26, 2023

On HP-dq3xxx, Alias opcode 0x06 is used, causing the parser to report an error.
@rw-vanc
Copy link
Contributor Author

rw-vanc commented Jan 26, 2023

I have an implementation that accepts Alias, but I have not written the as_integer etc. functionality. Let me know if you want this.

@IsaacWoods
Copy link
Member

Yep, we don't support aliases yet! Feel free to open a PR with your current impl - I always imagined they should be handled in the namespace layer (perhaps with a separate map, but not sure), so not sure this should need functionality in e.g. as_integer, but happy to review!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@IsaacWoods @rw-vanc