Skip to content
This repository has been archived by the owner on Mar 8, 2024. It is now read-only.

remove unused ASM #371

Merged
merged 4 commits into from
Nov 30, 2023
Merged

remove unused ASM #371

merged 4 commits into from
Nov 30, 2023

Conversation

bombcar
Copy link
Member

@bombcar bombcar commented Nov 29, 2023

This removes unused asm (either we config it off, or it literally just did nothing except print [UNUSED].

Full pack loads but someone who knows more about ASM may want to take a look. It's certainly a way of deciding what to ASM.

Also refactored a spelling error.

@bombcar bombcar requested a review from a team November 29, 2023 22:45
@bombcar bombcar marked this pull request as draft November 29, 2023 22:55
@bombcar bombcar marked this pull request as ready for review November 29, 2023 23:08
@bombcar
Copy link
Member Author

bombcar commented Nov 29, 2023

It's likely that this can be cleaned up further, the RWG ASM could go into RWG itself, if we determine what it is doing and why.

Copy link

Warning: 2 uncommitted changes
https://github.com/GTNewHorizons/bartworks/pull/372

@Caedis Caedis merged commit 32fb2a9 into master Nov 30, 2023
1 check passed
@Caedis Caedis deleted the nukeasm branch November 30, 2023 05:32
mitchej123 pushed a commit that referenced this pull request Mar 8, 2024
* remove unused ASM

* accidentally removed too much

* Remove RWG ASM

* spotlessApply

---------

Co-authored-by: Caedis <[email protected]>
Co-authored-by: GitHub GTNH Actions <unknown>
Former-commit-id: 5238bf977343cbfbf20b6e8b6c27ebeabd0ae1c9
mitchej123 pushed a commit that referenced this pull request Mar 8, 2024
* remove unused ASM

* accidentally removed too much

* Remove RWG ASM

* spotlessApply

---------

Co-authored-by: Caedis <[email protected]>
Co-authored-by: GitHub GTNH Actions <unknown>
Former-commit-id: ea5a2a8199749234e1e144c9f326ddd1b23e30c8
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants