New public LoadService#findFileToLoad method

The new method should be public, of course.

This probably won't get into 1.1.5, because we're in the home stretch right now. We could probably expose some of the basic stuff that's in there now. Can you have a look at trunk and propose a small change to make public what you want to get at?

Will do. I will submit a patch in the next few days.

I have come up with a workaround in my own code that obviates the need for a separate method that finds the file. It is a simplified version of LoadService#tryResourceFromLoadPathOrURL.

I think that with a bit of work I could separate the search from the load and write findFileToLoad, but if nobody else wants this feature then I'm not going to bother.

Well, nobody's requested it, so I guess it's up to you. I don't see any reason the search couldn't be made public, but if you have something that functions for now it may not be necessary to add.

Jim: Tell ya what...I'm going to mark this bug as resolved since recent versions of JRuby have encapsulated better the loading logic. See for example the smartLoad method on LoadService:

public boolean smartLoad(final String file) {
checkEmptyLoad(file);

SearchState state = new SearchState(file);
state.prepareRequireSearch(file);

try {
for (LoadSearcher searcher : searchers) {
if (searcher.shouldTrySearch(state))

else

}
} catch (AlreadyLoaded al)

return tryLoadingLibraryOrScript(runtime, state);
}

The entire searching process is now largely abstracted into a SearchState object passed through a series of LoadSearcher implementations. If you're still interested in a completely separate method, come up with something you see feel is appropriate and we'll include it. It should be easy to modify what we have now. Once you have something, reopen and we'll apply it.

Or, if you no longer have a need for this, we can leave it resolved as "incomplete".

On second thought, I'll add a basic version of the method you seek. You can submit a separate enhancement if you like.

It will basically look like this:

    public SearchState findFileForLoad(String file) throws AlreadyLoaded {
        SearchState state = new SearchState(file);
        state.prepareRequireSearch(file);

        for (LoadSearcher searcher : searchers) {
            if (searcher.shouldTrySearch(state)) {
                searcher.trySearch(state);
            } else {
                continue;
            }
        }

        return state;
    }

New API added in r9107.

So I can get the file that will be loaded from the SearchState instance, then? I don't see how. Perhaps I was asking for something impossible. All I want is the full file system path to the file that will get loaded when I "require 'foo'". If 'foo' isn't a plain Ruby file, then I don't care — return nil, return the class/jar/lib file, whatever.

Got it. Thank you. I didn't notice that the ivars were public.