Skip to content

Basic indenter for RARS#116

Open
ElTitoDG wants to merge 4 commits intorarsm:masterfrom
ElTitoDG:indenter
Open

Basic indenter for RARS#116
ElTitoDG wants to merge 4 commits intorarsm:masterfrom
ElTitoDG:indenter

Conversation

@ElTitoDG
Copy link

Implemented basic indenter for RARS
Solution to #23

@github-actions
Copy link

github-actions bot commented May 19, 2025

Test Results

12 files  ±0  12 suites  ±0   37s ⏱️ -10s
 8 tests ±0   8 ✅ ±0  0 💤 ±0  0 ❌ ±0 
24 runs  ±0  24 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit ccec0be. ± Comparison against base commit 814d573.

♻️ This comment has been updated with latest results.

Copy link
Collaborator

@privat privat left a comment

Choose a reason for hiding this comment

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

I have exception that pop

Exception in thread "AWT-EventQueue-0" java.lang.IllegalArgumentException: 2 > 1
	at java.base/java.util.Arrays.copyOfRange(Arrays.java:3804)
	at java.base/java.util.Arrays.copyOfRange(Arrays.java:3764)
	at rars.util.Indenter.indent(Indenter.java:57)
	at rars.util.Indenter.indentAssembly(Indenter.java:107)
	at rars.venus.VenusUI$24.actionPerformed(VenusUI.java:511)

it seems to fail when there is a loce label for instance

foo:
        bar

@ElTitoDG
Copy link
Author

ElTitoDG commented Jul 9, 2025

Resolve all the request by @privat
Before:
Screenshot 2025-07-09 at 19 24 49
After:
Screenshot 2025-07-09 at 19 25 16

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.

2 participants