-
Ge0rG
So I found another corner case logic bug where I was detecting "other device activity" before "is this a MUC reflection of my sent message", leading to suppression of notifications when I was active on a MUC.
-
Holger
Ge0rG: The intent was suppressing yaxim notifications while using the desktop client and the effect was suppressing yaxim notifications after sending MUC messages with yaxim?
-
Ge0rG
Holger: exactly
-
Ge0rG
The code processing the different kinds of messages, in the right order, is mere 170 LoC, but it's got really much implicit structural knowledge about processing dependencies in it.
-
Ge0rG
and now one mire item✎ -
Ge0rG
and now one more item ✏
-
Holger
Ge0rG: I wasn't aware yaxim shuts up when detecting other device activity. For how long?
-
Holger
Ge0rG: And do you also still treat carbons differently?
-
Ge0rG
Holger: that's a recent feature in the beta, not yet in the official releases. 144s, like Conversations on "short"
-
Ge0rG
Holger: no, there is no difference any more, except for carbons of ?OTR-stuff, which get dropped
-
Holger
Ge0rG: Ah. Sounds good.
-
Ge0rG
Hm. There is still some code to mark carbons as "already read", but I'm not sure if it's actually effective
-
Holger
144 sounds like a result of in-depth scientific research 🙂
-
Ge0rG
Uhm, yes.