search

mate-desktop / mate-notification-daemon

Daemon to display passive pop-up notifications
https://mate-desktop.org
GNU General Public License v2.0
30 stars 26 forks source link

Fix decoding the hints dictionary. #158

Closed raveit65 closed 4 years ago

raveit65 commented 4 years ago

Back port of https://github.com/mate-desktop/mate-notification-daemon/commit/8dad8a7

@flexiondotorg @vkareh @kbrenneman Please take a look at it. Commit from master didn't clean apply with a cherry-pick.

kbrenneman commented 4 years ago

Two g_variant_get_int32 calls that were removed in the original patch got left in this one.

If it makes things easier, I've also got a local branch with this fix on top of 1.22 -- I actually started there because it was easier to diagnose and test on my system. I can submit a PR with that if anyone wants.

vkareh commented 4 years ago

Yes, that would be helpful, thanks!

kbrenneman commented 4 years ago

Okay, I just posted #159.

raveit65 commented 4 years ago

Thanks, i will close this one.