Skip to content

Commit

Permalink
Updated todo
Browse files Browse the repository at this point in the history
  • Loading branch information
FedeDP committed Mar 17, 2018
1 parent f011725 commit 2157fcd
Showing 1 changed file with 2 additions and 24 deletions.
26 changes: 2 additions & 24 deletions TODO.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,33 +2,11 @@

## 1.0

### Drop on error callback?

- [x] drop modules_ctx_on_error (easy)
- [x] expand error defines (eg MOD_NOT_FOUND -2 etc etc) (easy)

### Generic

- [x] Add a module_pre_start function for each module (easy)
- [X] After module_stop -> STOPPED status (to prevent it being evaluated again after new event)
- [x] Add a m_update_fd(int fd, bool close_old) function to change fd (easy)
- [x] actually check if another mod with same name is already in context
- [x] MODULE and MODULE_CTX macro to take strings parameters
- [x] rename init in get_fd
- [x] rename module ctor in init() and dtor in destroy (they're static now)
- [ ] find a better alternative to $ctx_$name_pre_start for each module...
- [x] Check libmodule with valgrind again (last time it had no memleaks)
- [ ] find a better alternative to $ctx_$name_pre_start for each module pre_start function
- [ ] rename get_fd function (as same function should be used to eg: set module's initial subscriptions and initial state)
- [ ] Let users implement "non-pollable" modules, ie: modules that are not bound to a FD (but only receives pubsub message) -> ie: set RUNNING state anyway if certain FD is passed
- [x] module_is() to accept bitmask of states
- [x] protect needed functions by checking module's state

### Much more actor like?

- [x] implement tell and publish (per-ctx) (easy)
- [x] implement subscribe (mid)
- [x] add a pubsub type in msg_t (easy)

- [x] Update samples
- [ ] Update doc

**UPDATE DOC with changes until there (where to mention ctx##_##name_pre_start() function?) + RELEASE 1.0**
Expand Down

0 comments on commit 2157fcd

Please sign in to comment.