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.

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

"Unresolved symbol" in imports #55

Closed
profelis opened this issue Nov 7, 2014 · 8 comments
Closed

"Unresolved symbol" in imports #55

profelis opened this issue Nov 7, 2014 · 8 comments
Assignees
Labels

Comments

@profelis
Copy link

profelis commented Nov 7, 2014

import haxe.macro.Printer;

http://gyazo.com/fedd9e0937bb87ff937bc75b33492bde
image

my custom module

http://gyazo.com/451f69ba42871ce06dc9fae7237a07cc
image

TIR: STB-7298

@as3boyan
Copy link
Contributor

I think this means that parser chokes on those files.

https://github.com/profelis/hml/blob/master/src/hml/xml/Data.hx

@EBatTiVo
Copy link
Contributor

EBatTiVo commented Dec 5, 2014

Seems to be fixed on the ClassHierarchy branch now that #47 is fixed.

I tested the linked file, and made another that referenced it:

package hml.xml;
import hml.xml.Data; // <<--- Doesn't resolve, because it's not a class name.
import hml.xml.Node;  // A Class inside of Data.hx, resolves just fine.
class xmlTest {
    public function new() {
    }
}

Let me know if this didn't test what you reported.

@EBatTiVo
Copy link
Contributor

EBatTiVo commented Dec 5, 2014

Oh, yeah.

import haxe.macro.Printer;

resolved just fine, too.

@profelis
Copy link
Author

stil problem with

using haxe.macro.Tools;

@profelis
Copy link
Author

import haxe.macro.Expr.Position;

doesn't resolve too

@EBatTiVo EBatTiVo added the bug label Jan 29, 2015
@EBatTiVo
Copy link
Contributor

haxe.macro.Tools is a class that uses static extensions and typedefs to simplify importing and coding. I haven't found the root of the problem yet, though, since the file is on the standard classpath and should be found.

@EBatTiVo
Copy link
Contributor

Fixed at ClassHierarchy e5d7d7f

EBatTiVo added a commit that referenced this issue Feb 24, 2015
@EBatTiVo
Copy link
Contributor

@profelis I think this is fixed. Please re-open if your testing shows otherwise.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants