apache/brpc

Remove implementation specific function name of EventDispatcher

Open

#1894 aperta il 17 ago 2022

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)C++ (3936 fork)batch import
good first issuerefactor

Metriche repository

Star
 (16.226 star)
Metriche merge PR
 (Merge medio 6g 19h) (30 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Cerca tutte le occorrenze di AddEpollOut e altri nomi di funzioni specifici dell'implementazione nella classe EventDispatcher, quindi proponi e implementa nomi generici come AddHandler o RegisterEvent.
Tech stack
ccpp
Dominio
backendinfrastructure
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70