[
http://jira.codehaus.org/browse/MGWT-115?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=190632#action_190632
]
jieryn commented on MGWT-115:
-----------------------------
I run with this Util definition on quite large projects with 30-40 users at any
given time. I haven't seen personally or heard of any performance degradation
from static initialization. This is mitigated almost entirely because of
GWT.runAsync() code splitting, as the module will only be loaded when we need
it anyway.
Util having package visibility is just fine. We use it via something like
MyServiceAsync.Util.getInstance().invokeRpc(new AsyncCallback<....> {}); It
works across packages without any problem.
Finally, it isn't possible with today's tools to have these services generated
automatically and still have tools like Eclipse work properly. The
target/generated-sources isn't a default source directory, nor is it added by
maven-eclipse-plugin. This would require all my developers to add it manually,
which is a royal pain. Instead, we just run the generateAsync goal with an
output directory in the src/ folder.
> Generate cleaner (w/r/t checkstyle, findbugs, pmd) *Async Utility class
> -----------------------------------------------------------------------
>
> Key: MGWT-115
> URL: http://jira.codehaus.org/browse/MGWT-115
> Project: Maven 2.x GWT Plugin
> Issue Type: Improvement
> Affects Versions: 2.0
> Reporter: jieryn
> Priority: Minor
> Attachments: cleaner-generate-async-util-class.patch
>
>
> The Util class created during the generateAsync goal causes a lot of warnings
> for common static code analysis tools like Checkstyle, FindBugs, and PMD.
> Attached is a patch against r10656 which cleans up this utility class a bit.
> There still is work to be done where we are redundantly setting the service
> target if the service class already codes a @RemoteServiceRelativePath.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe from this list, please visit:
http://xircles.codehaus.org/manage_email
