search

godbus / dbus

Native Go bindings for D-Bus
BSD 2-Clause "Simplified" License
976 stars 225 forks source link

fix: fix panic for Variant sign is 's' ,but value is (*string) not st… #343

Open zhaolunxiang123 opened 1 year ago

zhaolunxiang123 commented 1 year ago

…ring

fix panic for Variant sign is 's' ,but value is (*string) not string

Log: fix panic for Variant sign is 's' ,but value is (*string) not string

guelfey commented 1 year ago

Fair point that pointers within Variants aren't handled correctly here. I'm not sure how they could be useful, but if creating them is possible, then formatting should be possible as well of course. Can you add code to make this generic for all pointer types by using reflect, and not just specific for string? Thanks.