coolguy
51e2568ba7
Units start here, with the part that needs no import graph: what a unit is called and where it must live. The parser only ever read a single identifier after `unit` and `import`, so `unit game.main;` and `import std.io;` were syntax errors -- which is why the dotted fixtures failed at the semicolon. It now reads a dotted path and stores it canonically, dots included, since that spelling is the unit's identity everywhere else. `import a.b as c;` parses too, with the alias on the node. resolve.c is the new pass between parsing and checking, for the questions that span files. It carries SPEC 8.1 so far: each path segment is ASCII lowercase, starts with a letter, continues with letters, digits or underscore, and is at most eight characters; and the dotted path must match the source path it was read from, so game.world.map has to come from game/world/map.fe. The source side is folded to lowercase before comparing, because a case-insensitive host must not let two spellings become two units. That rule then applied to the fixtures, which were not obeying it: 57 declared a unit name unrelated to their file, left over from the milestone directories, and eight had names too long to be legal. Both are now aligned -- the rule is worth having only if the tree follows it. units: badupper, badlong and unitbad pass. 138 -> 146 of 188. The rest of units/ needs the import graph, which is the next piece: resolution, cycles, bindings and visibility.
14 lines
269 B
Plaintext
14 lines
269 B
Plaintext
unit bad_clos;
|
|
|
|
struct FileLike {
|
|
handle: i32,
|
|
fn close(self: Self) -> !void { self.handle = 0; }
|
|
fn drop(self: &mut Self) { self.handle = 0; }
|
|
}
|
|
|
|
fn bad() -> !void {
|
|
let file: FileLike = FileLike{ handle: 7 };
|
|
try file.close();
|
|
file.close();
|
|
}
|