Countly / Countly/countly-sdk-cpp

No namespace

Offen
#33 0 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
C++
Sterne
34
Forks
25
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

Without a namespace, the contents of the header polute the global namespace. in particular, the using json is quite unfortunate

clients of this SDK need to wrap the #include <countly.hpp> in a namespace as a work-around

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne mit der Untersuchung des Headers countly.hpp und finde die using json-Deklaration sowie die anderen Inhalte auf oberster Ebene. Die Änderung ist abgeschlossen, wenn das Einbinden des Headers den globalen Namensraum nicht mehr verunreinigt und bestehende SDK-Clients weiterhin ohne den Namespace-Workaround kompiliert werden können.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp
Bereich
developer-experience
Issue-Typ
Bug
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.