Responses inline..<br><br><div class="gmail_quote">On Tue, Oct 23, 2012 at 7:38 PM, JFlower <span dir="ltr"><<a href="mailto:fiorenzino@gmail.com" target="_blank">fiorenzino@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<div><br></div><div>i started to find material to solve the bugs in object.</div><div><br></div><div><b>1) (FORGE-367) [SOLVED] question: how add import with wildcard and/or static:</b></div><div><br></div><div>see my comment: <a href="https://issues.jboss.org/browse/FORGE-367#comment-12728726" target="_blank">https://issues.jboss.org/browse/FORGE-367#comment-12728726</a></div>
<div><br></div><div>In this version we accept wildcard inside the className and we accept static option:</div><div><b>- javaClass.addImport("org.junit.Assert.*").setStatic(true);</b></div></blockquote><div><br>
I like the way you have solved this. <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><br></div><div>If you want we could add a new method:</div>
<div><b>- javaClass.addStaticImport("org.junit.Assert.*");</b></div></blockquote><div><br>No, I don't think this is necessary. We should be able to make the right decision.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div><br></div><div><b>2) (FORGE-563 - FORGE-424) question: creation of some WildcardImportResolver implementation</b> - where this resolver should find the complete className when we have some wildcard import?</div></blockquote>
<div><br>That's the tricky part.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div><br></div><div>- Could we use the current ClassLoader and find all classes in that?</div><div><br></div></blockquote><div><br>Well... the problem is that we can't use the Current classloader because that would be Forge's classloader. We need the classloader of the project, which is not actually classloaded, so we would need to classload the project (potentially very slow without an index.)<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div></div><div>- Could we create some index with all classes inside all mvn dependencies/modules jar [using jboss module capabilities? using tattletale? ]</div>
<div><br></div></blockquote><div>We need to do *something* - I'll leave it up to you to play with some ideas and see what works well or not!<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div></div><div>- Could we<b> </b>use jboss jandex to do that (also if jandex is <span style="background-color:rgb(246,246,246);font-family:Trebuchet,'Trebuchet MS',Arial,sans-serif;font-size:13px">processes Java annotations)</span>?</div>
<div><a href="http://javahowto.blogspot.de/2012/08/how-to-run-jboss-jandex.html" target="_blank">http://javahowto.blogspot.de/2012/08/how-to-run-jboss-jandex.html</a></div><div><a href="https://github.com/jbossas/jandex/blob/master/src/main/java/org/jboss/jandex/JarIndexer.java" target="_blank">https://github.com/jbossas/jandex/blob/master/src/main/java/org/jboss/jandex/JarIndexer.java</a></div>
<div><br></div><div><br></div></blockquote><div>Potentially.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div></div><div>Have you some idea?</div><div><br>
</div></blockquote><div>Nothing you haven't already mentioned :)<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div></div><div><br></div><div>Fiorenzo</div>
<br>_______________________________________________<br>
forge-dev mailing list<br>
<a href="mailto:forge-dev@lists.jboss.org">forge-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/forge-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/forge-dev</a><br>
<br></blockquote></div><br><br clear="all"><br>-- <br>Lincoln Baxter, III<br><a href="http://ocpsoft.org" target="_blank">http://ocpsoft.org</a><br>"Simpler is better."<br>