Complete test coverage of assembler

[app-scheme79asm.git] / lib / App / Scheme79asm.pm

diff --git a/lib/App/Scheme79asm.pm b/lib/App/Scheme79asm.pm
index f76b834ea6c463ae7b4a03dce6baaf097612a358..0333b700dd9361da79b962c70a0ca48b073c016c 100644 (file)
--- a/lib/App/Scheme79asm.pm
+++ b/lib/App/Scheme79asm.pm
@@ -57,17 +57,7 @@ sub process {
 
        $addr = $self->process($addr) if ref $addr eq 'ARRAY';
        die 'Addr of toplevel is not atom: ', Dumper($addr), "\n" unless scalarp($addr);
-
        my ($comment_type, $comment_addr) = ($type, $addr);
-
-       unless (looks_like_number $addr) { # is symbol
-               unless (exists $self->{symbols}{$addr}) {
-                       $self->{symbols}{$addr} = $self->{nsymbols};
-                       $self->{nsymbols}++;
-               }
-               $addr = $self->{symbols}{$addr}
-       }
-
        die 'Computed addr is not a number: ', Dumper($addr), "\n" unless looks_like_number $addr;
 
        if (!looks_like_number $type) {
@@ -114,15 +104,13 @@ sub new {
        $args{addr_bits} //= 8;
        $args{freeptr} //= 6;
        $args{memory} //= [0, 0, (1<<$args{addr_bits}), (1<<$args{addr_bits}), 0, 0, 0];
-       $args{symbols}{T} = 2;
-       $args{nsymbols} = 3;
        $args{comment} = ['(cdr part of NIL)', '(car part of NIL)', '(cdr part of T)', '(car part of T)', '(free storage pointer)', '', '(result of computation)'];
        bless \%args, $class
 }
 
 sub print_binary16 {
        my ($self, $fh) = @_;
-       $fh //= \*STDOUT;
+       $fh //= \*STDOUT; # uncoverable condition right
 
        die "addr_bits + type_bits >= 16\n"if $self->{addr_bits} + $self->{type_bits} > 16;
 
@@ -135,7 +123,7 @@ sub print_binary16 {
 
 sub print_verilog {
        my ($self, $fh) = @_;
-       $fh //= \*STDOUT;
+       $fh //= \*STDOUT; # uncoverable condition right
 
        my $bits = $self->{type_bits} + $self->{addr_bits};
        my $index_length = length $#{$self->{memory}};
@@ -150,7 +138,10 @@ sub print_verilog {
                }
                my $spaces = ' ' x ($bits + 5 - (length $val));
                $index = sprintf $index_format, $index;
-               say $fh "mem[$index] <= $val;$spaces // $comment"
+
+               print $fh "mem[$index] <= $val;";
+               print $fh "$spaces // $comment" if defined $comment;
+               print $fh "\n";
        }
 
 }
@@ -192,7 +183,7 @@ The SIMPLE processor expects input in a particular tagged-pointer
 format. This module takes a string containing a sequence of
 S-expressions. Each S-expression is a list of one of three types:
 
-C<(tag value)>, for example C<(symbol nil)>, represents a value to be
+C<(tag value)>, for example C<(symbol 2)>, represents a value to be
 put in memory (for example a number, or a symbol, or a variable
 reference).
 
@@ -295,16 +286,6 @@ C<freeptr>.
 The initial comments for memory entries. C<< $comment->[$i] >> is the
 comment for C<< $memory->[$i] >>.
 
-=item symbols
-
-The initial symbol map, as a hashref from symbol name to the index of
-that symbol. Defaults to C<< {T => 2} >>.
-
-=item nsymbols
-
-The number to give to the "next" symbol (default 3, because T is
-defined to be 2).
-
 =back
 
 =item $asm->B<parse>(I<$string>)