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

[Lincoln Baxter, III]

Okay, that makes sense, but when the time comes, I would allow Jandex to
index annotations as well, since that is extremely important as we go
forward with CDI and similar annotation-driven programming models. The
Class index we generate could be used for more than just wildcards.

Thoughts?
~Lincoln

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

> Hi Lincoln,
>
>
> jandex does more things than i need:
>
> for my test, i removed some methods inside jandex ((e.g.:i don't care
> about annotations inside classes - i removed the generation of jandex file
> index).
>
>
> I started using the default jandex code, but to reduce the time of
> scanning my jars inside test case, i modified the jandex source code.
>
>
> It's dirty code..but the understand how jandex works, i needed that!
>
>
>  have a nice we,
>
>
> fiorenzo
>
> 2012/10/26 Lincoln Baxter, III <lincolnbaxter at gmail.com>
>
>> 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."
>>
>> _______________________________________________
>> 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/e6d80e24/attachment.html