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

Develop #90

Merged
merged 31 commits into from Oct 10, 2021
Merged

Develop #90

merged 31 commits into from Oct 10, 2021

Conversation

alshapton
Copy link
Owner

@alshapton alshapton commented Oct 10, 2021

  • Improved Assembler code by simplifying
  • Simplified some low-level instruction code
  • Corrected code styling
  • Updated RDx and RDR functions
  • Completed WMP, WRx and WRR functions

alshapton and others added 29 commits October 9, 2021 21:11
This commit fixes the style issues introduced in 72acb7d according to the output
from autopep8.

Details: https://deepsource.io/gh/alshapton/Pyntel4004/transform/85d0c56a-51b2-493b-a6d0-f20ec74f2745/
This commit fixes the style issues introduced in 2abe70a according to the output
from autopep8.

Details: https://deepsource.io/gh/alshapton/Pyntel4004/transform/3ebd3f98-4985-44f0-b7b8-4072712f4977/
Also - all tests now using convert_to_absolute_address
This commit fixes the style issues introduced in 6ba9051 according to the output
from autopep8.

Details: https://deepsource.io/gh/alshapton/Pyntel4004/transform/6dc0148b-b7cd-4815-81cb-a9514eb945cb/
This commit fixes the style issues introduced in 5fd9dc2 according to the output
from autopep8.

Details: https://deepsource.io/gh/alshapton/Pyntel4004/transform/f7afde61-c2cd-4e69-9959-bf2ce452cf37/
@pep8speaks
Copy link

pep8speaks commented Oct 10, 2021

Hello @alshapton! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 305:80: E501 line too long (80 > 79 characters)
Line 306:80: E501 line too long (81 > 79 characters)

Comment last updated at 2021-10-10 20:44:04 UTC

@alshapton alshapton merged commit b7eb74b into main Oct 10, 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