Can't build linearscan.rs

[indutny]

https://github.com/indutny/linearscan.rs

It was working with pre 0.7 rust, but name lookup doesn't seem to be working now :(

[emberian]

@indutny Is this still relevant? Do you need help?

[indutny]

I'll be able to tell you once I'll port all code to new rust :) Iterators has changed dramatically.

[indutny]

@cmr it seems to be still broken... I've ported my code to use new iterators but I still get:

test/emulator.rs:210:15: 210:25 error: unresolved name `Graph::new`.
test/emulator.rs:210   let mut g = ~Graph::new();

[indutny]

Any help would be welcome!

[emberian]

@indutny I'll take a poke at it later today, thanks for following up.

[indutny]

@cmr thank you!

[alexcrichton]

Turns out this is the problem:

use linearscan::Graph;

mod linearscan {

  mod graph {
    pub struct Graph { block_id: uint, }
    impl Graph {
      pub fn new() -> Graph { fail!() }
    }
  }

  impl Graph {}
}

pub fn main() {
  Graph::new();
}

Has this error message:

$ make
rustc  --test test/runner.rs -o run-tests
test/runner.rs:16:2: 16:12 error: unresolved name `Graph::new`.
test/runner.rs:16   Graph::new();
                    ^~~~~~~~~~
error: aborting due to previous error
make: *** [run-tests] Error 101

Which is kind of right, but also kind of weird. I think that the problem is that in linearscan::api you have impl<...> Graph<...> while in linearscan::graph you have a separate impl<...> Graph<...>. I think that this falls under the category of "resolve surprises", but I'm not entirely sure what's going on here.

In the meantime, if you merge the two impl blocks for Graph, your code should get past resolve at least.

[indutny]

Apparently @alexcrichton's comment fixes the issue, thanks!