apache/brpc

Remove implementation specific function name of EventDispatcher

Open

Aperta il 17 ago 2022

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

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

Tech stack
cpp
Dominio
backend
Tipo issue
refactor
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 hours
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
stale
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
mostly clear
Prerequisiti
C++ programmingEventDispatcher component
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60
Direzione di ricerca
Examine the EventDispatcher class in the source code, likely in `src/brpc/event dispatcher.cpp`. Identify all occurrences of the function `AddEpollOut` and understand its platform specific context. Propose a generic name such as `AddPollableFd` or `AddEvent`, then rename the function and update all references. Check the issue comments for any prior discussion on naming conventions or preferred alternatives.