stan-dev/stan

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

Offen

#2.577 geöffnet am 10.07.2018

Auf GitHub ansehenContributor Guide
 (13 Kommentare) (0 Reaktionen) (1 zugewiesene Person)C++ (387 Forks)batch import
code cleanupgood first issue

Repository-Metriken

Stars
 (2.761 Sterne)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Identifizieren Sie alle Vorkommen des Musters `if (msg.str().length() > 0) logger.info(msg);` im Code. Bewerten Sie zwei Ansätze: (1) Ändern Sie die info() Methode, um leere Nachrichten zu ignorieren, oder (2) kapseln Sie das Muster in einer Hilfsfunktion. Implementieren Sie die gewählte Lösung konsistent und aktualisieren Sie die Tests entsprechend.
Tech Stack
cpp
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60