Wizzup | I also had discord 'working' to some degree with the purple plugin, but didn't test it much at all since I don't use it | 02:21 |
---|---|---|
Wizzup | lol I am so used to fremantle on n900 that I type 'sudo gainroot' on leste frequently | 14:45 |
Wizzup | I will try to get signal and telegram working in some super basic fashion as well, even though iiuc those need one to have some other smartphone device, but just for the heck of it | 15:15 |
sicelo | Telegram doesn't require another device | 15:48 |
bencoh | for a new account it does, unfortunately | 15:48 |
sicelo | Nice @ sudo gainroot 😉 | 15:48 |
sicelo | bencoh: needs it for? | 15:49 |
sicelo | Unless something changed lately, they just need to be able to send you an sms (or voice call) | 15:50 |
sicelo | No reason this can't be a Leste phone | 15:50 |
bencoh | sicelo: they changed it "recently" (~a year ago?) sadly | 15:52 |
bencoh | at least that's what I remember reading from their website () | 15:52 |
sicelo | I don't remember that at all. Will check | 15:54 |
sym8298 | heya finally found u guys :-) | 16:07 |
gnarface | \o/ | 16:10 |
* sicelo runs away from sym8298 | 16:12 | |
bencoh | you scared him :( | 16:30 |
freemangordon | Wizzup: conversations seems to leak sqlite db file handles | 16:57 |
freemangordon | perhaps it opens new connection every time wihtout closing th old one | 16:57 |
freemangordon | Wizzup: who unrefs el from https://github.com/maemo-leste/conversations/blob/master/src/lib/rtcom.cpp#L8 | 17:02 |
freemangordon | also, I don't understand why is this https://github.com/maemo-leste/conversations/blob/master/src/lib/rtcom.h#L11 | 17:08 |
freemangordon | this violates layering and whatnot | 17:08 |
freemangordon | Wizzup: dsc_: is conversations multi-threaded? | 17:15 |
Wizzup | hi | 17:24 |
freemangordon | hi | 17:24 |
Wizzup | afaik the queries are unref'd properly | 17:25 |
freemangordon | but not rtcomel | 17:25 |
Wizzup | this leaking of handles, you see this in /proc/*/fd ? | 17:25 |
Wizzup | hm | 17:25 |
freemangordon | yes, it does | 17:25 |
Wizzup | the rtcom el should be global | 17:25 |
Wizzup | I'll fix this | 17:25 |
freemangordon | https://pastebin.com/m4FmwDjG | 17:26 |
freemangordon | I know, but it can;t be | 17:26 |
freemangordon | because you use it outside of rtcom.cpp :) | 17:26 |
Wizzup | sicelo: I am unable to make an account without an android/ios device fwiw | 17:26 |
Wizzup | I'll take a look | 17:26 |
Wizzup | @ rtcom leak | 17:26 |
freemangordon | Wizzup: do you want me to refactor that code? (rtcom) | 17:27 |
Wizzup | feel free if you want to | 17:27 |
Wizzup | I need to go to the shop first I think | 17:27 |
freemangordon | ok, I'll do it | 17:27 |
freemangordon | not now though | 17:27 |
freemangordon | do you want me to make PR, or I can commit directly? | 17:27 |
Wizzup | just commit | 17:28 |
freemangordon | ok | 17:29 |
freemangordon | Wizzup: again, is conversations multi-threaded? | 17:31 |
Wizzup | no threads are explicitly made | 17:31 |
freemangordon | ok | 17:31 |
Wizzup | I don't know what tpqt, osso-abook, or others do | 17:31 |
freemangordon | the question was - shall I use mutexes to protect shared data | 17:32 |
freemangordon | seems no | 17:32 |
Wizzup | right now I don't think it is necessary | 17:32 |
Generated by irclog2html.py 2.17.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!