Skip to content

Conversation

@paragonie-scott
Copy link
Collaborator

No description provided.

@defuse
Copy link
Owner

defuse commented Oct 16, 2015

How about we just map class names to paths?

$map = array(
  'Crypto' => 'src/Crypto.php',
  'Core' => 'src/Core.php',
 ...
)

@paragonie-scott
Copy link
Collaborator Author

Yeah, that's probably more sensible. Does #116 cover this already? (If so, I will close. If not, I will fix.)

@defuse
Copy link
Owner

defuse commented Oct 17, 2015

#116 doesn't do it yet, feel free to do it if you want.

@paragonie-scott
Copy link
Collaborator Author

Okay, I believe this is ready to go. :)

@defuse
Copy link
Owner

defuse commented Oct 17, 2015

Thanks. Going to attempt rebasing this onto my other WIP branch :-)

@defuse
Copy link
Owner

defuse commented Oct 17, 2015

Can't figure that out right now, I'll do it tomorrow. :-)

@defuse
Copy link
Owner

defuse commented Oct 17, 2015

(If you happen to get to it first, I added some new files, and I intend to change it to do $classmap[$relative_class] for efficiency).

@paragonie-scott
Copy link
Collaborator Author

Dammit, I did not mean to push that commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants