neatorobito / Salut

A nice library for working with WiFi P2P service discovery on Android.
MIT License
297 stars 88 forks source link

Few features #30

Closed adroitandroid closed 7 years ago

adroitandroid commented 7 years ago

-> Salut.java hasn't been changed much apart from the abstract method for serialization, it shows the entire file changed since file encoding has been changed

neatorobito commented 7 years ago

Thank you for your work. I will push a release in a bit.

abselt commented 7 years ago

after public abstract class Salut

call not work Salut network = new Salut(dataReceiver, serviceData, new SalutCallback() { @Override public void call() { } });

adroitandroid commented 7 years ago

Right, you'll have to implement the serialize method to use the serializer library of your choice

On Fri, Jul 14, 2017 at 3:11 PM, abselt notifications@github.com wrote:

after public abstract class Salut

call not work Salut network = new Salut(dataReceiver, serviceData, new SalutCallback() { @Override public void call() { } });

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/markrjr/Salut/pull/30#issuecomment-315317884, or mute the thread https://github.com/notifications/unsubscribe-auth/AIDhjkhOmcR4vG62NcpRiRe0pCMae3PHks5sNzfXgaJpZM4ODk-i .

abselt commented 7 years ago

thanks im not too much into java i lerned android directly do i need to convert my activity to abstract also and will that cause a danger to other code or app perfurmance counsidering that i will keep using the default serializer

adroitandroid commented 7 years ago

When your Android studio complaints about unimplemented, implement it and since you're using LoganSquare, you'll need to return LoganSquare.serialize(<method's argument>) within it

On 15-Jul-2017 2:40 AM, "abselt" notifications@github.com wrote:

thanks im not too much into java i lerned android directly do i need to convert my activity to abstract also and will that cause a danger to other code or app perfurmance counsidering that i will keep using the default serializer

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/markrjr/Salut/pull/30#issuecomment-315468666, or mute the thread https://github.com/notifications/unsubscribe-auth/AIDhjhf6OocblOfZ0jYGVcfagKByTfp0ks5sN9lRgaJpZM4ODk-i .

abselt commented 7 years ago

still cant understand how to make things right way thank you any way i will stick to my previous copy of the lib untel i learn that serialize thing but i hope you take a look at issue #29 may you help us Im sure every one is facing this

adroitandroid commented 7 years ago

@markrjr is there code for a sample app in which I can show the new implementation @abselt I'm not sure what the problem in #29 is, I had tried the underlying Android APIs directly before and for me they were unstable because they didn't guarantee discovery. That's when I came up with my own library Near, that doesn't rely on the Android APIs at all

abselt commented 7 years ago

you are right it is more due to android API not this lib but still for example there is many actions in broadcast receiver for WIFI-direct are not implemented here i tried to make use of them but didnt make it alot better.