Hi Dan, Thank you for your response. I took a look at the change, and using MsgArg instead of AboutObjectDescription looks intentional. I think it's too late to question the intent for the 16.10 release, but I may investigate the intent later and bring this issue again after the 16.10 release.
Thanks, George On Wed, Aug 10, 2016 at 11:17 AM, Daniel Mihai <[email protected]> wrote: > Hi George, > > > > It looks like the change from AboutObjectDescription to MsgArg was > intentional, here: https://git.allseenalliance.org/gerrit/#/c/2193/. If > you haven’t looked already at these old changes, please take a look, and > then we can discuss the possibility of additional changes in the Core WG > meeting. > > > > Dan > > > > *From:* [email protected] [mailto: > [email protected]] *On Behalf Of *George Tang > *Sent:* Monday, August 8, 2016 12:30 PM > *To:* allseen-core <[email protected]> > *Subject:* [Allseen-core] AboutProxy > > > > Hi, > > > > Should AboutProxy.GetObjectDescriptions be returning an > AboutObjectDescription? > > > > AboutProxy.GetObjectDescriptions returns a MsgArg object. This is the > AboutProxy of the C++ bindings. > > > > The AboutProxy.getObjectDescriptions in the Java bindings returns an > AboutObjectDescriptions array. > > > > I noticed in the AboutProxy.cc that it includes AboutObjectDescription, > but never uses it. I think someone may have forgotten to convert a MsgArg > to AboutObjectDescription before committing. > > > > So should AboutProxy return an AboutObjectDescription instead? > > > > Thanks, > > George >
_______________________________________________ Allseen-core mailing list [email protected] https://lists.allseenalliance.org/mailman/listinfo/allseen-core
