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

JFlower fiorenzino at gmail.com
Fri Oct 26 10:16:37 EDT 2012


Dirty hacking code!

:-)


2012/10/26 Lincoln Baxter, III <lincolnbaxter at gmail.com>

> 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 at gmail.com> 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 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
>>>
>>>
>>
>> _______________________________________________
>> 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/02edc9c2/attachment.html 


More information about the forge-dev mailing list