FXML and Classloading

Greg Brown greg.x.brown at oracle.com
Sun Jun 10 04:47:19 PDT 2012

Hi Tom,
As I mentioned, while I understand the motivation, I'm still not convinced that it is necessary. But yes, please go ahead and file a JIRA ticket. Thanks.

On Jun 9, 2012, at 1:28 PM, Tom Schindl wrote:

> So where do we go from here? We do agree that there's something to be
> done? So I should file a JIRA-Ticket?
> Tom
> Am 31.05.12 18:11, schrieb Greg Brown:
>>> The problem is that if I'm using composition I don't get the correct
>>> classloader
>> Sure you do. You just need to propagate it to your nested builder factory.
>>> I think you should not set a default classloader at all but let the
>>> type-classloader be the default - it is the classloader used by the
>>> FXMLLoader while constructing the Object graph.
>> OK. I *think* the only real issue here is whether or not we truly need to pass a class loader to JavaFXBuilderFactory. The only place where we actually need the class loader in JavaFXBuilderFactory is when we are trying to load the generated builder class for a given type. If we assume that the builder class will be loaded by the same class loader that was used to load the original type, then you are right - we don't need to explicitly pass a class loader to the JavaFXBuilderFactory constructor. However, I'm not sure that is a safe assumption. But I also don't think it is a major inconvenience to have to do so - if you need to use a custom class loader in your app, you just need to make sure you pass it to your JavaFXBuilderFactory instance.
> -- 
> B e s t S o l u t i o n . a t                        EDV Systemhaus GmbH
> ------------------------------------------------------------------------
> tom schindl                 geschäftsführer/CEO
> ------------------------------------------------------------------------
> eduard-bodem-gasse 5-7/1   A-6020 innsbruck     fax      ++43 512 935833
> http://www.BestSolution.at                      phone    ++43 512 935834

More information about the openjfx-dev mailing list