Am 7. März 2012 17:26 schrieb Richard S. Hall (Commented) (JIRA)
<[email protected]>:
>
>    [ 
> https://issues.apache.org/jira/browse/FELIX-3376?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13224477#comment-13224477
>  ]
>
> Richard S. Hall commented on FELIX-3376:
> ----------------------------------------
>
> I committed these patches with only minor modifications.
>
> One thing I thought about was we could potentially replace the custom 
> BundleTracker with the standard OSGi BundleTracker...although I'm torn on 
> this a little since our custom BundleTracker is simpler and it does provide a 
> good example of how to create such a thing.

I agree with you about BundleTracker. We should try to reuse stuff
that's already out there, even if it is a little more complex. People
will tend to reuse the code they see in examples. So it's better to
show them "the right way".

Speaking of reusing stuff: Java 6 has a nice solution for creating the
FrameworkFactory
(http://docs.oracle.com/javase/6/docs/api/java/util/ServiceLoader.html).
Looking at the bunch of code I've ridden to parse the
org.osgi.framework.launch.FrameworkFactory file, that would make
things easier. If you don't feel to comfortable about Java 6, I would
at least recommend to add a comment point users to the javadocs above.

What do you think?

Benedikt

>
> Please close this issue if you are satisfied. Thanks.
>
>> Update extenderbased SimpleShapes example to use Java 5 Features
>> ----------------------------------------------------------------
>>
>>                 Key: FELIX-3376
>>                 URL: https://issues.apache.org/jira/browse/FELIX-3376
>>             Project: Felix
>>          Issue Type: Improvement
>>            Reporter: Benedikt Ritter
>>            Priority: Trivial
>>         Attachments: FELIX-3376-extenderbased.circle.txt, 
>> FELIX-3376-extenderbased.host.txt, FELIX-3376-extenderbased.square.txt, 
>> FELIX-3376-extenderbased.triangle.txt
>>
>>
>> This follows up on FELIX-3375.
>
> --
> This message is automatically generated by JIRA.
> If you think it was sent incorrectly, please contact your JIRA 
> administrators: 
> https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
> For more information on JIRA, see: http://www.atlassian.com/software/jira
>
>

Reply via email to