uber/piranha

PiranhaJava - Doesn't add required space while removing parentheses

Open

#37 ouverte le 14 mai 2020

Voir sur GitHub
 (6 commentaires) (0 réactions) (0 assignés)Java (2 143 stars) (178 forks)batch import
good first issuelegacy-java

Description

Piranha changed return(true); to returntrue;. This is sort of related to #35 - in the sense that PiranhaJava doesn't validate the parens/spaces of the output generated.

This error probably would not have occured if the code had been Google Java formatted.

However, I thought I will bring it to your attention, since this is a case of Piranha introducing a "bug" instead of erring on the side of caution, like #35

Guide contributeur

Stack technique
java
Domaine
toolingdeveloper experience
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Javastring manipulation
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75
Direction de recherche
Investigate the code responsible for removing parentheses in flag conditions, particularly where the return statement's space is omitted. Look at the logic that generates the output string after transformation. Review issue #35 for related validation discussions. The fix likely involves ensuring a space is inserted between 'return' and the opening parenthesis when the parenthesis is removed.