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

Alternative prefixes don't work #6

Closed
jchristgit opened this issue May 16, 2019 · 1 comment
Closed

Alternative prefixes don't work #6

jchristgit opened this issue May 16, 2019 · 1 comment
Assignees
@jchristgit
Labels
bug Something isn't working component:invokers Issues related to command invoker behaviour & implementations wontfix This will not be worked on
Milestone
v0.3.0

Comments

@jchristgit
Copy link
Owner

My endlessly smart idea of looking up the prefix at compile time didn't work, because when compiling with other prefixes it doesn't work. We should look up the prefix at runtime instead, since an ETS read is cheap anyways.
@jchristgit jchristgit added bug Something isn't working component:invokers Issues related to command invoker behaviour & implementations labels May 16, 2019
@jchristgit jchristgit added this to the v0.3.0 milestone May 16, 2019
@jchristgit jchristgit self-assigned this May 16, 2019
@jchristgit
Copy link
Owner Author

Cannot be looked up elsewhere due to the nature of binary matching.

@jchristgit jchristgit added the wontfix This will not be worked on label May 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jchristgit jchristgit

Labels
bug Something isn't working component:invokers Issues related to command invoker behaviour & implementations wontfix This will not be worked on
Projects
None yet
Milestone
v0.3.0
Development

No branches or pull requests
1 participant
@jchristgit