Re: [forge-dev] questions about wildcard import [FORGE-367 - FORGE-563 - FORGE-424]

Dirty hacking code! :-) 2012/10/26 Lincoln Baxter, III <lincolnbaxter(a)gmail.com&gt; I may be a bit ignorant about Jandex. But what is a FlowerIndex? ;) Lol. > > ~Lincoln > > > On Thu, Oct 25, 2012 at 9:08 PM, JFlower <fiorenzino(a)gmail.com&gt; wrote: > >> Hi, >> >> i created this jira: https://issues.jboss.org/browse/FORGE-699 >> >> my code: https://github.com/fiorenzino/jandex/tree/FORGE-699 >> >> Comments are always welcome!! >> >> Fiorenzo >> >> ps: my first jandex hacking https://gist.github.com/3956397 >> >> *numbers: 66 libraries - scanned in 1458 milliseconds - 15036 classes >> (with subclasses)* >> >> >> >> 2012/10/24 Lincoln Baxter, III <lincolnbaxter(a)gmail.com&gt; >> >> Responses inline.. >>> >>> On Tue, Oct 23, 2012 at 7:38 PM, JFlower <fiorenzino(a)gmail.com&gt; wrote: >>> >>>> Hi, >>>> >>>> i started to find material to solve the bugs in object. >>>> >>>> *1) (FORGE-367) [SOLVED] question: how add import with wildcard >>>> and/or static:* >>>> >>>> see my comment: >>>> https://issues.jboss.org/browse/FORGE-367#comment-12728726 >>>> >>>> In this version we accept wildcard inside the className and we accept >>>> static option: >>>> *- javaClass.addImport("org.junit.Assert.*").setStatic(true);* >>>> >>> >>> I like the way you have solved this. >>> >>>> >>>> If you want we could add a new method: >>>> *- javaClass.addStaticImport("org.junit.Assert.*");* >>>> >>> >>> No, I don't think this is necessary. We should be able to make the >>> right decision. >>> >>> >>>> >>>> *2) (FORGE-563 - FORGE-424) question: creation of >>>> some WildcardImportResolver implementation* - where this resolver >>>> should find the complete className when we have some wildcard import? >>>> >>> >>> That's the tricky part. >>> >>> >>>> >>>> - Could we use the current ClassLoader and find all classes in that? >>>> >>>> >>> 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.) >>> >>> >>>> - Could we create some index with all classes inside all mvn >>>> dependencies/modules jar [using jboss module capabilities? >>>> using tattletale? ] >>>> >>>> We need to do *something* - I'll leave it up to you to play with some >>> ideas and see what works well or not! >>> >>> >>>> - Could we* *use jboss jandex to do that (also if jandex is processes >>>> Java annotations)? >>>> http://javahowto.blogspot.de/2012/08/how-to-run-jboss-jandex.html >>>> >>>> https://github.com/jbossas/jandex/blob/master/src/main/java/org/jboss/jan... >>>> >>>> >>>> Potentially. >>> >>> >>>> Have you some idea? >>>> >>>> Nothing you haven't already mentioned :) >>> >>> >>>> >>>> Fiorenzo >>>> >>>> _______________________________________________ >>>> forge-dev mailing list >>>> forge-dev(a)lists.jboss.org >>>> https://lists.jboss.org/mailman/listinfo/forge-dev >>>> >>>> >>> >>> >>> -- >>> Lincoln Baxter, III >>> http://ocpsoft.org >>> "Simpler is better." >>> >>> _______________________________________________ >>> forge-dev mailing list >>> forge-dev(a)lists.jboss.org >>> https://lists.jboss.org/mailman/listinfo/forge-dev >>> >>> >> >> _______________________________________________ >> forge-dev mailing list >> forge-dev(a)lists.jboss.org >> https://lists.jboss.org/mailman/listinfo/forge-dev >> >> > > > -- > Lincoln Baxter, III > http://ocpsoft.org > "Simpler is better." > > _______________________________________________ > forge-dev mailing list > forge-dev(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/forge-dev > > _______________________________________________ forge-dev mailing list forge-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/forge-dev