Closed MrCrazys closed 1 year ago
Thanks, I'll pull it manually later.
We can't rename methods because it will break all programs our students are using for now, I only do this on major version bumps (i.e. 5.0 to 6.0) so I can't pull this commit entirely.
Alright, that's okay. Just a personal preference I suppose. It doesn't really matter. Just happy to contribute!
newList(Object...)
as the name was misleading (it isn't clear that it creates a new ArrayList; you would have to look at the source code to see that). I have renamed it tonewArrayList(Object...)
. This makes it more consistent with thenewHashMap(Object, Object)
method, which has a good name that makes it obvious what type of Map is being created.newSet(Object...)
as the name was misleading (it isn't clear that it creates a new HashSet; you would have to look at the source code to see that). I have renamed it tonewHashSet(Object...)
. This makes it more consistent with thenewHashMap(Object, Object)
method, which has a good name that makes it obvious what type of Map is being created.newList(Object...)
method and replaced it with thenewArrayList(Object...)
method.Map.entrySet().forEach()
withMap.forEach()
).