arduino / arduino/ArduinoCore-API

[Stream.h] make MultiTarget and findMulti public

Offen
#105 2 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
C++
Sterne
306
Forks
150
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

Why MultiTarget and findMulti in Stream.h are not public members of Stream? Their implementation is by no means trivial and they seem relatively safe, so why hide this functionality from users?
I found it very useful e.g. for parsing text commands coming from serial port. I'm sure there's a lot of other cases where it may help.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Beginne in Stream.h, indem du die Deklarationen und Implementierungen von MultiTarget und findMulti lokalisierst, und prüfe anschließend, wie ihre Sichtbarkeit derzeit definiert ist. Erledigt ist die Aufgabe, wenn beide Member öffentlich zugänglich sind, ohne ihr bestehendes Verhalten zu ändern; prüfe die umgebende Stream API und die verfügbaren Tests auf Kompatibilität.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp
Bereich
api, embedded-iot
Issue-Typ
Feature
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
38/100

Neue Issues direkt in Ihr Postfach

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