GNOME Bugzilla – Bug 792301
TrackerNotifier should be more pedantic about integer sizes when parsing them out of GVariants
Last modified: 2018-01-23 14:39:08 UTC
The GraphUpdated signal is supposed to have "sa(iiii)a(iiii)" parameters. The GVariant type string "i" refers to a 32-bit signed integer. Therefore, gint32 is a much safer bet than gint, whose size is not guaranteed across all platforms.
Created attachment 366457 [details] [review] libtracker-sparql: Be more careful about integer sizes
Comment on attachment 366457 [details] [review] libtracker-sparql: Be more careful about integer sizes Makes sense given the current state of things. It's not the only wrong type length along the chain, but we can't easily move to int64_t enabled GraphUpdated without API breaks or horrible migration anyway...
Comment on attachment 366457 [details] [review] libtracker-sparql: Be more careful about integer sizes Thanks, Carlos.