Filippo Gentile

Results 220 comments of Filippo Gentile

I've rebased on current master

Cannot build it anymore with MinGW + Clang linker: ``` ld.lld: error: undefined symbol: TrainVehicleListTableModel::isListedProperty(std::basic_string_view) >>> referenced by C:/Qt_Project/traintastic/server/src/train\trainvehiclelist.cpp:129 >>> CMakeFiles/traintastic-server.dir/src/train/trainvehiclelist.cpp.obj:(TrainVehicleList::propertyChanged(BaseProperty&)) ld.lld: error: undefined symbol: TrainVehicleListTableModel::propertyChanged(BaseProperty&, unsigned int) >>> referenced...

Found the issue but I need suggestions. Z21's `ClientKernel` internally calls: (NOTE: lambda capture `this` to access `this->m_logId`) ``` EventLoop::call( [this, msg=toString(message)]() { Log::log(m_logId, LogMessage::D2001_TX_X, msg); }); ``` When communication...

In this particolar case we could just capture a copy of log ID and not capture `this` at all. But in other lambdas a reference to `this` is needed and...

This problem is partially fixed inside #59 but other lambdas passed to `EventLoop::call(...)` are capturing `this` so we need to change them also. Other interfaces might have the same issue....

I cannot reproduce anymore!

Quick comparison with #59 approach (not fully accurate): ## **LocoCache** approach PROS: - Less memory footprint when number of loco is low and many messages are sent - Can extract...

> p.s. As the Z21 implementation is receiving many improvements, it seems fair to me that you add a Copyright line for your name. (Don't know how this normally works...this...

I now have built a Z21 with ESP32 and did some initial tests. Pending reply queue is the way to go. We will keep some bits of my old `LocoCache`...