Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Classloaders #2241

Closed
wants to merge 2 commits into from
Closed

Classloaders #2241

wants to merge 2 commits into from

Commits on Nov 25, 2014

  1. remove extra whitespaces

    @mkristian
    mkristian committed Nov 25, 2014
    Copy the full SHA
    ba2fb1a View commit details
    Browse the repository at this point in the history

  2. clean up classloader type hierachy 

    * only the classloader runtime.getJRubyClassLoader needs to have
  all the extra features. all other classloaders just needs to
  be able to define classes and do not need the extra overhead of
  loading and finding resources and adding jar files during runtime.
* the OneShotClassloader is only used with runtime.getJRubyClassLoader
  so ensure this relation via types
* the JRubyClassLoader is used at various places also as "flag via instanceof".
  keep this and all other uses of JRubyClassLoader as it was before
* replace on use of OneShotClassloader with JRubyClassLoader to check if it possible
  to load byte code at runtime
    @mkristian
    mkristian committed Nov 25, 2014
    Copy the full SHA
    3ce1329 View commit details
    Browse the repository at this point in the history