node-dbus icon indicating copy to clipboard operation
node-dbus copied to clipboard

Fix issue with multiple return values

Open tjbcg opened this issue 7 years ago • 3 comments

This fixes issue #132 by allowing multiple return values

tjbcg avatar Jul 26 '18 22:07 tjbcg

breaks test case: https://github.com/Shouqun/node-dbus/blob/master/test/client-call-object.test.js#L17

Shouqun avatar Jul 28 '18 14:07 Shouqun

Adding a method which returns multiple values is also not accounted for, though multiple in arguments are supported. It should be corrected here.

https://github.com/Shouqun/node-dbus/blob/7c87b54cb5a107e6b3d3c610cc1f64cd2d26c4ec/lib/service_interface.js#L235-L239

Currently the inspection for out arguments results in this:

<arg direction="out" type="undefined"/>

AndreasBackx avatar Jan 23 '19 15:01 AndreasBackx

@Shouqun any chance this could get fixed/looked at? There are perhaps some more things that need a look at?

AndreasBackx avatar Jan 23 '19 15:01 AndreasBackx