search

jll63 / openmethods.d

Open multi-methods for the D language. OPEN! Multi is cool. Open is great.
44 stars 8 forks source link

Using `module foo.bar` breaks compilation. #6

Closed ghost closed 7 years ago

ghost commented 7 years ago

The following piece doesn't compile:

/* source/fileName.d */
module app.someModule;

import openmethods;
mixin(registerMethods);

void main()
{
}

A superficial fix is to put parentheses around %ss in registerMethods, but this just fixes compilation, and I'm not familiar with the source enough to tell if that's the only place that needs changes (that's why I'm opening an issue instead of making a pull request with the fix)

jll63 commented 7 years ago

Good catch, and your fix is correct. You send me a PR? That will make you an official contributor ;-)