Skip to content

Conversation

@tbw777
Copy link
Contributor

@tbw777 tbw777 commented Feb 15, 2023

Removed new operations from code like this:
MessageFormat.format(formater, new Object[]{ value });

Removed new operations from code like this:
MessageFormat.format(formater, new Object[]{ value });
Removed new operations from code like this:
MessageFormat.format(formater, new Object[]{ value });
Removed new operations from code like this:
MessageFormat.format(formater, new Object[]{ value });
Removed new operations from code like this:
MessageFormat.format(formater, new Object[]{ value });
Removed new operations from code like this:
MessageFormat.format(formater, new Object[]{ value });
Removed new operations from code like this:
MessageFormat.format(formater, new Object[]{ value });
Removed new operations from code like this:
MessageFormat.format(formater, new Object[]{ value });
Removed new operations from code like this:
MessageFormat.format(formater, new Object[]{ value });
@tbw777 tbw777 changed the title Removeed unnessasary new Object calls Removeed unnessasary new Object calls on MessageFormat Feb 15, 2023
@tbw777 tbw777 changed the title Removeed unnessasary new Object calls on MessageFormat Removeed unnessasary new Object calls on MessageFormat. Multicommit Feb 15, 2023
@tbw777 tbw777 changed the title Removeed unnessasary new Object calls on MessageFormat. Multicommit Removeed unnecessary new Object calls on MessageFormat. Multicommit Feb 15, 2023
@tbw777 tbw777 changed the title Removeed unnecessary new Object calls on MessageFormat. Multicommit Removed unnecessary new Object calls on MessageFormat. Multicommit Feb 15, 2023
@vieiro
Copy link
Contributor

vieiro commented Feb 18, 2023

Every time we use varargs, the Java compiler creates an array to hold the given parameters.

As a consequence, you're not really "Removed unnecessary new Object calls on MessageFormat" , the compiler will add these for you.

I can't see enough value on this PR.

@vieiro vieiro added do not merge Don't merge this PR, it is not ready or just demonstration purposes. syntax sugar labels Feb 18, 2023
@tbw777 tbw777 closed this Feb 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge Don't merge this PR, it is not ready or just demonstration purposes. syntax sugar

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants