stan-dev/stan

Clean up code: `.str().length()`

Ouverte

#2 577 ouverte le 10 juil. 2018

Voir sur GitHubGuide contributeur
 (13 commentaires) (0 réaction) (1 personne assignée)C++ (387 forks)batch import
code cleanupgood first issue

Métriques du dépôt

Stars
 (2 761 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Summary:

There are many instances of code that looks like:

if (msg.str().length() > 0)
  logger.info(msg);

We should either:

  • have the info() method ignore empty messages in general
  • wrap the call in a function

Additional Information:

Originally from this PR comment: https://github.com/stan-dev/stan/pull/2570#discussion_r201151286

Current Version:

v2.17.1

Guide contributeur

Direction de recherche
Identifiez toutes les occurrences du motif `if (msg.str().length() > 0) logger.info(msg);` dans la base de code. Évaluez deux approches : (1) modifier la méthode info() pour ignorer les messages vides, ou (2) encapsuler le motif dans une fonction d'assistance. Implémentez la solution choisie de manière cohérente et mettez à jour les tests en conséquence.
Stack technique
cpp
Domaine
backend
Type d'issue
Refactorisation
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.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60