On 3/7/12 17:12 , Benedikt Ritter wrote:
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).

I wouldn't go so far as calling it "nice", but it is a solution... ;-)

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?

In the end, I don't care. The original approach was taken to allow it to work on older JVMs. Since we are breaking that now, it doesn't really matter any more if we use ServiceLoader.

-> richard


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