Skip to content

Help SezPoz a little#4

Closed
dscho wants to merge 1 commit into
masterfrom
context-class-loader
Closed

Help SezPoz a little#4
dscho wants to merge 1 commit into
masterfrom
context-class-loader

Conversation

@dscho

@dscho dscho commented May 21, 2013

Copy link
Copy Markdown
Contributor

SezPoz uses the current Thread's context class loader to discover our
plugins. But if the DefaultPluginFinder cannot be loaded by said class
loader, it is safe to assume that SezPoz cannot find the plugins wanted
by the caller, either.

So let's not use the current Thread's context class loader in that case,
but the best we can do in that case: the class loader that found the
DefaultPluginFinder.

Signed-off-by: Johannes Schindelin johannes.schindelin@gmx.de

SezPoz uses the current Thread's context class loader to discover our
plugins. But if the DefaultPluginFinder cannot be loaded by said class
loader, it is safe to assume that SezPoz cannot find the plugins wanted
by the caller, either.

So let's not use the current Thread's context class loader in that case,
but the best we can do in that case: the class loader that found the
DefaultPluginFinder.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
@dscho

dscho commented Dec 14, 2013

Copy link
Copy Markdown
Contributor Author

Since we do not use SezPoz anymore, this pull request is obsolete.

@dscho dscho closed this Dec 14, 2013
@dscho dscho deleted the context-class-loader branch December 14, 2013 02:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant