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

[Lincoln Baxter, III]

And why did you fork the Jandex repo? :p

On Fri, Oct 26, 2012 at 10:16 AM, JFlower <fiorenzino at gmail.com> wrote:

> 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
>>
>>
>
> _______________________________________________
> 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."
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/forge-dev/attachments/20121026/7e7bad2b/attachment-0001.html