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

QOL: --output= defaults to .class #15

Merged
merged 2 commits into from
Apr 20, 2024

Conversation

Kreijstal
Copy link
Contributor

it's tiresome to write .class everytime, specially if its current directory, by default it takes the filename and adds .class to it.

Copy link

Unit Test Results

87 tests  ±0   86 ✔️ ±0   0s ⏱️ ±0s
12 suites ±0     1 💤 ±0 
12 files   ±0     0 ±0 

Results for commit f5f9b3f. ± Comparison against base commit 1246e37.

This pull request removes 47 and adds 28 tests. Note that renamed tests count towards both.

		iadd
		iload a
		iload b
		ireturn	
	.code: {
	.parameters: {a, b}
	type: .enum java/lang/TargetType, METHOD
	value: "java/lang/HelloWorld", 
	}
…
me.darknet.assembler.EscapeTest ‑ [12] 







me.darknet.assembler.EscapeTest ‑ [5] @ParameterizedTest
@ValueSource(
    strings = {
        "epic\u0020obfuscated\u0020name",
        "{ \"Hello World\", type: \"java/lang/HelloWorld\" }",
        "0 -10 10f 10.16F 10.161616D 10L 0xDEADBEEF 0E10",
        ".class public java/lang/HelloWorld"
    }
)

me.darknet.assembler.SampleCompilerTest$RoundTrip ‑ [10] Example-variables.jasm
me.darknet.assembler.SampleCompilerTest$RoundTrip ‑ [11] Example-inner-class.jasm
me.darknet.assembler.SampleCompilerTest$RoundTrip ‑ [12] Example-int-remainder.jasm
me.darknet.assembler.SampleCompilerTest$RoundTrip ‑ [13] Example-int-subtraction.jasm
me.darknet.assembler.SampleCompilerTest$RoundTrip ‑ [14] Example-invokedynamic.jasm
me.darknet.assembler.SampleCompilerTest$RoundTrip ‑ [15] Example-invokedynamic-short.jasm
me.darknet.assembler.SampleCompilerTest$RoundTrip ‑ [16] Example-int-multi.jasm
me.darknet.assembler.SampleCompilerTest$RoundTrip ‑ [17] Example-int-multiplication.jasm
…

@jumanji144 jumanji144 merged commit fb31612 into jumanji144:master Apr 20, 2024
3 checks passed
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