urho icon indicating copy to clipboard operation
urho copied to clipboard

More bindings (types required for binary serialization) and Urho update

Open Alan-FGR opened this issue 7 years ago • 10 comments

Hello there. I was implementing binary serialization but when I was halfway into it I realized the AbstractFile (which inherits both Serializer and Deserializer) and superclasses aren't bound yet, on the native side they provide type-specific write/read methods that are piped into a single method for that purpose. Since they are many methods I wonder if you guys are planning to update Urho soon and perhaps you could also point the binding generator at those classes required for binary serialization, they're not complex classes but of course there's the problem of multiple inheritance. Worst case scenario I guess it could be reimplemented fully managed by passing pointer and size in the callback.

Alan-FGR avatar Feb 09 '18 19:02 Alan-FGR

Also, lemme take this opportunity to ask another question: I see that you need to add the callbacks in Node.cpp, are you guys using some specific program to add those there or is it fully manual (and rebased/merged directly with git+some diff merger)?

Alan-FGR avatar Feb 09 '18 19:02 Alan-FGR

By the way, on a side note, luckily Urho creates a VectorBuffer for each Component when serializing what makes it pretty easy and safe to use for managed components.

Alan-FGR avatar Feb 09 '18 19:02 Alan-FGR

I need this too

troll-kadabra avatar Feb 09 '18 21:02 troll-kadabra

@Alan-FGR yes, the callbacks to managed side in Node are hand written so you can add yours if you need them :-) I'll try to bind AbstractFile

EgorBo avatar Feb 15 '18 19:02 EgorBo

@EgorBo yes, no problem in that. The only thing really missing is the binding since manually doing that is too much work, the serialization step itself I can do, so it would be very helpful if that class could be bound :). Thank you. P.S.: any plans to update UrhoSharp to a newer Urho version? :P

Alan-FGR avatar Feb 15 '18 19:02 Alan-FGR

@Alan-FGR after I push what I have now to nuget - it usually takes less than an hour to update all bindings and test everything (unless urho has some SDL updates)

EgorBo avatar Feb 15 '18 19:02 EgorBo

@EgorBo no hurry. Thank you.

Alan-FGR avatar Feb 15 '18 20:02 Alan-FGR

Hi. Is this done?

troll-kadabra avatar Apr 23 '18 14:04 troll-kadabra

Hi. Any new?

troll-kadabra avatar Apr 30 '18 00:04 troll-kadabra

not news on this?

troll-kadabra avatar Sep 05 '18 21:09 troll-kadabra