Skip to content
This repository has been archived by the owner on Jun 18, 2023. It is now read-only.

Brace being set on a new line for classes #5

Open
ChloeDawn opened this issue May 25, 2019 · 5 comments
Open

Brace being set on a new line for classes #5

ChloeDawn opened this issue May 25, 2019 · 5 comments

Comments

@ChloeDawn
Copy link

Braces for classes are set on a new line rather than inline like other braces. This bug was experienced through Enigma. An example of what I am describing can be seen below.

public class FooBar
{
	public void foobar() {
		if (Baz.condition()) {
			for (int i = 0; i < 10; ++i) {
			}
		}
	}
}
@Gegy
Copy link

Gegy commented May 25, 2019

I think this would be considered an Enigma issue -- the brace style is configurable through JavaFormattingOptions. This is the default but maybe Enigma should change the setting?

@ChloeDawn
Copy link
Author

Oh, I assumed this was an issue at decompiler level, my bad. Can you transfer this to the Enigma repo?

@Gegy
Copy link

Gegy commented May 25, 2019

I don't believe I have the ability to do that, but could recreate the issue if needed?

@ChloeDawn
Copy link
Author

Oh, I think only members/collaborators can do it, nvm. If you had permissions you would see this option in the right-hand sidebar:
screenshot

@Runemoro
Copy link
Contributor

This was fixed a while ago in FabricMC/Enigma#160.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants