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

JFlower fiorenzino at gmail.com
Thu Oct 25 21:08:24 EDT 2012


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 at gmail.com>

> Responses inline..
>
> On Tue, Oct 23, 2012 at 7:38 PM, JFlower <fiorenzino at gmail.com> 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/jandex/JarIndexer.java
>>
>>
>> Potentially.
>
>
>> Have you some idea?
>>
>> Nothing you haven't already mentioned :)
>
>
>>
>> Fiorenzo
>>
>> _______________________________________________
>> forge-dev mailing list
>> forge-dev at 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 at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/forge-dev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/forge-dev/attachments/20121026/160db3ca/attachment-0001.html 


More information about the forge-dev mailing list