Task #2298
closed
Rename nfd::EventId to nfd::scheduler::EventId
Added by Junxiao Shi almost 10 years ago.
Updated almost 10 years ago.
Description
nfd::EventId is an alias of ndn::EventId, which is an opaque type representing ID of a scheduled event on a Scheduler.
However, bare EventId is not readable, because there's no hint of its association with Scheduler.
This Task is to change EventId to scheduler::EventId, to emphasize its association with nfd::scheduler.
There is only one scheduler, why do you need additional association? Just "EventId" seem enough to represent that it is something about scheduled events.
I prefer keeping code as is.
- Subject changed from Eliminate nfd::EventId to Rename nfd::EventId to nfd::scheduler::EventId
- Estimated time changed from 1.50 h to 0.50 h
In general, I'm against pointless changes of the existing code. I agree there could be some confusion with EventEmitter, but there shouldn't be confusing with Signal and Connections.
If you really want to make this change, I wouldn't object.
- Estimated time changed from 0.50 h to 1.00 h
It's not a "pointless change". It's an improvement which makes code more readable.
- Status changed from New to In Progress
- Assignee set to Junxiao Shi
- Status changed from In Progress to Code review
- % Done changed from 0 to 100
- Status changed from Code review to Closed
Also available in: Atom
PDF
Updated by 
Updated by