Turbo Macro Pro X support improvements#113
Closed
laplace7 wants to merge 29 commits intorolandshacks:masterfrom
Closed
Turbo Macro Pro X support improvements#113laplace7 wants to merge 29 commits intorolandshacks:masterfrom
laplace7 wants to merge 29 commits intorolandshacks:masterfrom
Conversation
These are constants for representing PETSCII codes that can't be typed on PC
To avoid seeing all label uses as macro calls in instructions, such as `lda#const`, macros can only be invoked at the start of a line or after a label (maybe other cases?). The loose detection of acme macros (`+...`) impacts operator usage; acme also should only be identifying macro calls before instructions.
It's hard to choose what scopes to use, out-of-the-box syntax colours for assembly are pretty poor for assembly in VSCode.
* Added schema for file project-config.json. That way code completion
and documentation is available when users edits the file in vscode.
* Improved tmpx build support:
- respect project property "main" for main source file
- added property "cwd" for the current working directory,
where tmpx is executed. This is necessary so that
.includes relative to the main.asm file work correctly.
* Added Turbo Macro Pro specific files:
- grammar
- language
- snippets
* Added TMP 06 Compatibility settings:
- Max Code Line Length
- Max Comment Line Length
* The languageId for assembly source files is now "tmpx" if
the toolkit is "tmpx".
* Added a simple tmpx code formatter.
It only indents code lines that have no leading label.
Author
|
I have tested all examples (acme, basic, basic_tsb, cc65, kick, llvm, oscar64, tmpx) under Linux Mint
I have added problem matches for llvm, cc65, tmpx as no problems were shown |
Author
|
|
laplace7
changed the title
Owner
|
Hi. While this all is impressive effort - the size of those PRs are a real challenge given the limited time for me to review. |
Owner
|
FYI: I currently doing some major refactoring which will soon come with 2.7.2+. Maybe that would be a better baseline for functional changes. |
Author
|
That sounds good. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR extends the branch from PR #99
and documentation is available when users edits the file in vscode.
where tmpx is executed. This is necessary so that
.includes relative to the main.asm file work correctly.
the toolkit is "tmpx".
It only indents code lines that have no leading label.
Comment lines (starting with a semi-colon) are not changed.