Hi Otávio,
I'd say no in this case. If the performance is equivalent I'd leave it
alone.
Cheers.
Bruno Baptista
https://twitter.com/brunobat_
On 05/12/18 15:27, Otávio Gonçalves de Santana wrote:
Hello everyone, I'm studying the code, mostly the container module. It
seems that in the master we support Java 8.
My question: Does make sense to create PRs to update some APIs to a modern
way?
I mean, e.g.:
for (int i = 0; i < rules.length; i++) {
rules[i].validate(appModule);
}
To
for (ValidationRule rule : rules) {
rule.validate(appModule);
}
Ps: That does not impact in performance, just does the code
cleaner/readable and more natural to maintainer IMHO.
