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

[JFlower]

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