hypfvieh / dbus-java

Improved version of java DBus library provided by freedesktop.org (https://dbus.freedesktop.org/doc/dbus-java/)
https://hypfvieh.github.io/dbus-java/
MIT License
185 stars 73 forks source link

Fixed sending lists over the bus in an async manner #38

Closed rm5248 closed 5 years ago

rm5248 commented 5 years ago

When sending lists over the bus using callMethodAsync, if one of the parameters is a list(e.g. subclass of java.util.list), but not a java.util.list directly, it will fail to send over the bus due to the library trying to interpret the ArrayList as a param to the function.

Also added a test to ensure that this behavior does work correctly.

I'm not exactly sure why this is not needed for Structs. It may also be needed for subclasses of java.util.Map, but that is not a use-case for me.

hypfvieh commented 5 years ago

Thanks for your patch. I did some cleanup and merged it.