apache/brpc

Remove implementation specific function name of EventDispatcher

Offen

#1.894 geöffnet am 17.08.2022

Auf GitHub ansehenContributor Guide
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (3.936 Forks)batch import
good first issuerefactor

Repository-Metriken

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

Beschreibung

Some of the member functions of EventDispatcher are implementation specific, for example AddEpollOut.

We need to change that specific function name and adopt more general name then we can easily support brpc to more platforms.

Contributor Guide

Research-Richtung
Suchen Sie nach allen Vorkommen von AddEpollOut und anderen implementierungsspezifischen Funktionsnamen in der EventDispatcher Klasse und schlagen Sie dann allgemeine Namen wie AddHandler oder RegisterEvent vor und implementieren Sie diese.
Tech Stack
ccpp
Domain
backendinfrastructure
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-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++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70