google/benchmark

Dynamically initialized globals are problematic

Offen

#233 geöffnet am 02.06.2016

Auf GitHub ansehenContributor Guide
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (1.539 Forks)batch import
help wanted

Repository-Metriken

Stars
 (7.968 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

we have a few global non-PODs in tests. these should be removed.

https://google.github.io/styleguide/cppguide.html#Static_and_Global_Variables

Contributor Guide

Research-Richtung
Identifizieren Sie alle nicht POD globalen Variablen in Testdateien, verstehen Sie deren Initialisierungsabhängigkeiten und refaktorieren Sie sie in lokale oder funktionsstatische Variablen, um Probleme mit der dynamischen Initialisierung zu vermeiden. Befolgen Sie die Empfehlungen des Google C++ Style Guides.
Tech Stack
cpp
Domain
testingbackend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
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.
75