search

tkem / fsmlite

Lightweight finite state machine framework for C++11
MIT License
156 stars 25 forks source link

Drop basic_row #21

Closed tkem closed 5 years ago

tkem commented 5 years ago

It's kind of awkward to use, and with C++17 serves no real purpose any more. Also think about renaming mem_fn_row to simply row on C++11, should be reasonably forward-compatible with the C++17 implementation (except for overloaded member functions).

tkem commented 5 years ago

Renaming (and therefore effectively dropping mem_fn_row) turns out to be a burden in practice, since this will effectively break application code when moving to C++17. So both should be kept, and basic_row probably too for now.