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

Suggestion: Auto-gray Permgen option for Java 8+ or allow individual activation of each memory option #1552

Open
kotoroshinoto opened this issue Apr 22, 2016 · 13 comments

Comments

@kotoroshinoto
Copy link

new versions of java don't HAVE a permgen option anymore, and the VM puts a complaint when you supply it. No harm done, its more of an extremely minor irritation than anything super important..

Also wouldn't hurt to have an advanced panel with some of the other memory options, and some more common garbage collector and thread model options in a checklist.

Obviously that is achievable already with the arguments box, but it'd be neat.

@RPoor
Copy link

RPoor commented May 20, 2016

I second this

@jikuja
Copy link

jikuja commented May 20, 2016

Also note that java 9 will exit and print error message if -XX:MaxPermSize or -XX:PermSize are used. Which makes this more then minor at some point in the future.

Am I confused with my reactions? Yes, there is no need to post +1 messages anymore. Just use correct reaction.

@peterix
Copy link
Member

peterix commented May 20, 2016

Just don't touch the option and it won't be passed to java at all.

@kotoroshinoto
Copy link
Author

its being passed in my instances, I do set the memory options, if need be, put that one under the control of a separate checkbox.

@peterix
Copy link
Member

peterix commented May 20, 2016

No. If permgen is set to 64, it is not passed.

@kotoroshinoto
Copy link
Author

that is a REALLY awkward method to control it, nonobvious to a general user who might want to edit their values.

@peterix
Copy link
Member

peterix commented May 20, 2016

permgen will eventually be forgotten... and I want to completely change how the game is launched. So any effort put into this right now is IMHO wasted.

@kotoroshinoto
Copy link
Author

well if the system is going to be redesigned anyway, this issue will be resolved.

Perhaps you could insert a tooltip or bit of text below the input box explaining that a value of 64 won't be passed, to allow users to avoid runtime complaints.

@kotoroshinoto
Copy link
Author

that'd be less complex than changing the launcher logic

@peterix
Copy link
Member

peterix commented May 20, 2016

The launcher logic simply has to change because of java 9, broken display drivers on windows, ability to run more than one instance, ability to run server instances, support for launch via java agent, disconnecting running instances from MultiMC so MultiMC can be closed and reopened while keeping track of what's running, and so on...

There is a LOT that has to be done with launching.

Not saying that it won't end up with some simple probe to disable that UI field anyway... but I'd rather work on the big changes first.

@kotoroshinoto
Copy link
Author

oh I know, I meant as a more immediate measure, as adding a textbox is considerably less invasive and doesn't have to change the logic at the moment.

@peterix
Copy link
Member

peterix commented Nov 21, 2018

This is not completely done. It's done only for the instance overrides.

@mombof
Copy link

mombof commented Jun 9, 2019

Is there currently a work-around for this?
Also this prevents users from being able to get past the initial java setup after install of MCMulti, if they change xmX or xms values while in that same window.

@peterix peterix closed this as completed Jun 1, 2022
@phit phit reopened this Jun 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants