Re: [PATCH spice-gtk v2 0/5] Fix introspection method bindings

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Nov 30, 2017 at 09:33:32AM -0500, Marc-André Lureau wrote:
> Hi
> 
> ----- Original Message -----
> > On Tue, Sep 19, 2017 at 10:33:56PM +0800, Matthew Francis wrote:
> > > Various C functions have not been named correctly to be picked up as bound
> > > methods under introspection. This renames them to ensure correct binding.
> > 
> > I'm very late to the party, but had you considered using (rename-to
> > NEWNAME) annotations rather than wholesale renaming like in this series?
> > This would have meant a lot less changes, with no impact on existing
> > code bases.
> 
> I wished that would work, but that doesn't seem supported. Feel free to find a solution before the next release ;)

It indeed behaves oddly. If I add a (rename-to) for a non-existing
symbol, then it complains during gir generation:
channel-main.h:115: Warning: SpiceClientGLib: Can't find symbol 'spice_main_channel_update_display3' referenced by "rename-to" annotation
but if I add the new symbol, then the (rename-to) seems to do something,
but is pointless as the new symbol will show up in the .gir even if we
don't add any (rename-to). I don't understand what it's meant to be used
for ;)

Christophe

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]