Re: gwt-inerop-utils preview

2016-06-03 Thread Hristo Stoyanov
Paul, I am also hesitating a bit too ... I see some people that I respect do go for the whole enchilada with no regrets. Here is a JS AP I that requires 4-argument callb

Re: gwt-inerop-utils preview

2016-06-03 Thread Paul Stockley
Jens, I can see your point, about the usability when you get many arguments. Originally I was going to kind of mirror the Java JDK and not go more than 2. Personally, I think it is a lot better to define custom interfaces when possible. To avoid having hundreds of class files I have started

Re: gwt-inerop-utils preview

2016-06-03 Thread Paul Stockley
Jens, -- You received this message because you are subscribed to the Google Groups "GWT Users" group. To unsubscribe from this group and stop receiving emails from it, send an email to google-web-toolkit+unsubscr...@googlegroups.com. To post to this group, send email to google-web-toolkit@googl

Re: gwt-inerop-utils preview

2016-06-02 Thread Jens
> > If adding a couple of more interfaces is code smell, take a look at this > > ... > Yeah I know... > Also, take a look at JS APIs like this > - are we all suppos

Re: gwt-inerop-utils preview

2016-06-02 Thread Hristo Stoyanov
Thanks Paul, Let me use the github issues forgiving you feedback, this forum is not the best way. On Thursday, June 2, 2016 at 2:17:10 PM UTC-7, Paul Stockley wrote: > > I will add 4 and 5 arg interfaces. I think I can include something > equivalent with your above code. I already have getters/s

Re: gwt-inerop-utils preview

2016-06-02 Thread Hristo Stoyanov
Jen, If adding a couple of more interfaces is code smell, take a look at this ... Also, take a look at JS APIs like this - are we all supposed to define the same 4/5-ar

Re: gwt-inerop-utils preview

2016-06-02 Thread Jens
> I will add 4 and 5 arg interfaces. > I can already smell the code smell.. void myAsyncFunction(Func5 onSuccess, Func5 onFailure); IMHO its fine to just have Func1 and for anything else chose a concrete name for a concrete functional interface so you have a place to document all these para

Re: gwt-inerop-utils preview

2016-06-02 Thread Paul Stockley
I will take a look at the putnami-gradle-plugin. I noticed Steffan's didn't seem to be supported. Thanks for the offer on migration. The projects are a bit in flux at the moment so probably better off not submitting a pull request until I have landed all the gwt-interop-utils changes. -- You r

Re: gwt-inerop-utils preview

2016-06-02 Thread Paul Stockley
I will add 4 and 5 arg interfaces. I think I can include something equivalent with your above code. I already have getters/setters and is defined functions for objects. All you would need to do is pass $wnd into these. Let me have a play around and see what I can come up with. On Thursday, June

Re: gwt-inerop-utils preview

2016-06-02 Thread Hristo Stoyanov
... also, I noticed you are using the Steffan's GWT Gradle plugin. I did encounter subtle issues with it lately (see gihub issue threads) and realized it is no longer supported. I highly recommend switching to this one . It is simpler, provides

Re: gwt-inerop-utils preview

2016-06-02 Thread Hristo Stoyanov
Thanks Paul, I was about to start releasing similar code . It does not make sense to do the same thing twice and obviously such utility code is necessary. So, I will use your project as it see

gwt-inerop-utils preview

2016-06-02 Thread Paul Stockley
I have committed an initial version of gwt-interop-utils. https://github.com/GWTReact/gwt-interop-utils This library provides some common utilities for working with JsInterop e.g. 1) Object Literal support 2) Shared JSON compatible structures 3) Common functional interfaces 4) JSON utilities 5)