Skip to content

Conversation

@Ronald-FM
Copy link

@Ronald-FM Ronald-FM commented Nov 20, 2025

Este Pull Request aplica um conjunto de melhorias orientadas pelo PMD no módulo
jetty-core/jetty-server, com o objetivo de aumentar a clareza, consistência e
manutenibilidade do código, sem alterar comportamento ou lógica funcional.

As alterações incluídas neste PR são:

• Remoção de qualificadores totalmente qualificados desnecessários
(UnnecessaryFullyQualifiedName), substituindo-os por imports adequados.

• Simplificação de expressões que continham parênteses redundantes
(UselessParentheses), mantendo exatamente a mesma precedência e sem qualquer
impacto semântico.

• Reestruturação de condicionais aninhadas quando possível
(CollapsibleIfStatements), tornando o fluxo mais direto e reduzindo níveis de
indentação — apenas onde a combinação era realmente segura.

• Revisão de falsos positivos relacionados a AutoLock em try-with-resources.
O PMD sinalizou alguns padrões como UnusedLocalVariable, porém estes objetos
são essenciais para o mecanismo de liberação automática do lock. Nestes casos,
não houve modificação de código.

O conjunto de refatorações foi executado de forma conservadora, preservando
total compatibilidade com a implementação existente.

Processo de build e validação:

  1. mvn -DskipTests clean install
  2. mvn -DskipTests pmd:pmd

Após a aplicação das mudanças, o projeto compila normalmente e nenhuma
regressão foi identificada. As regras do PMD foram reexecutadas para confirmar
a redução dos smells reportados.

@joakime
Copy link
Contributor

joakime commented Nov 20, 2025

PR does not compile.
PR also fails checkstyle tests.
PR also fails antlr input checks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants