danvratil / harbour-sailslack

Unofficial Slack client for Sailfish OS
GNU General Public License v3.0
4 stars 4 forks source link

Fix longstanding milliseconds bug #41

Closed b100dian closed 1 year ago

b100dian commented 1 year ago

The timestamp seconds to milliseconds conversion was adding the part after the . (which is a 6 figure number) and which was polluting the seconds part by adding as much as 16 minutes (max 999 seconds) to the timestamp.

Since Qt is not having a microsecond datetime, for now I just use the first 3 figures after the dot, and drop the rest.

(Previous text) (not tested yet on master, only on feature/storage branch, I have a dbus build problem but pretty sure this is 5+years bug in that code)

I ended up running manually into src qdbusxml2cpp -c SailslackAdaptor -a sailslack_adaptor.h:sailslack_adaptor.cpp harbour.sailslack.xml.

Tested on master now too