Opened 10 years ago

Closed 10 years ago

Last modified 8 years ago

#39 closed defect (fixed)

Emit an error if modules are recursive.

Reported by: benl Owned by: erikd
Priority: normal Milestone:
Component: Compilation Driver Version:
Keywords: Cc:


We don't handle recursive module yet, so should at least emit an error.

Currently, the compilation driver does into an endless loop.

Change History (4)

comment:1 Changed 10 years ago by erikd

  • Owner set to erikd
  • Status changed from new to assigned

comment:2 Changed 10 years ago by erikd

Test is T39-RecursiveModule?.

Four modules; Main.ds, A.ds, B.ds and C.ds.

Main imports B which imports C which imports A which imports B creating a cycle: A -> B -> C -> A.

comment:3 Changed 10 years ago by erikd

  • Resolution set to fixed
  • Status changed from assigned to closed

Fixed in the following set of patches:

Fri Nov 13 23:13:28 EST 2009 Erik de Castro Lopo <erikd@…>

Fri Nov 13 22:34:31 EST 2009 Erik de Castro Lopo <erikd@…>

  • Fix #39 : Error out on mutually recursive module import.

Fri Nov 13 20:18:40 EST 2009 Erik de Castro Lopo <erikd@…>

  • Panic on mutually recursive modules instead of blowing the stack.

Still need to convert the panic to an error.

comment:4 Changed 8 years ago by benl

  • Milestone 0.1.4 deleted

Milestone 0.1.4 deleted

Note: See TracTickets for help on using tickets.