[m-dev.] for review: bytecode interpreter

Fergus Henderson fjh at cs.mu.OZ.AU
Sat Dec 9 10:52:12 AEDT 2000


On 08-Dec-2000, Levi Cameron <l.cameron2 at ugrad.unimelb.edu.au> wrote:
> 
> Estimated hours taken: About a week
>
>   Now that I have started on the actual interpreter instead of
>   just support code, this is the beginnings of the new bytecode
>   interpreter. Probably just over half the code is the same or a
>   superficially modified revision of the old one.
> 
> mb_bytecode
> Almost identical. Only real change is made MB_read_cstring not use
> any static variables
>
> mb_disasm
> Again, code very similar.

Could you please post cvs diffs for the files which haven't
changed much?

Also in future please use our standard layout for review messages:
	- "Estimated hours taken" field should preferably be machine-parsable,
	  so e.g. "40 (roughly)" is better than "About a week"
	- File names should include directory name and extension
	  (e.g. bytecode/mb_bytecode.c)
	- ":" after each file name
	- Comments for each file should be indented one tab
	- Comments at the top should not be indented

-- 
Fergus Henderson <fjh at cs.mu.oz.au>  |  "I have always known that the pursuit
                                    |  of excellence is a lethal habit"
WWW: <http://www.cs.mu.oz.au/~fjh>  |     -- the last words of T. S. Garp.
--------------------------------------------------------------------------
mercury-developers mailing list
Post messages to:       mercury-developers at cs.mu.oz.au
Administrative Queries: owner-mercury-developers at cs.mu.oz.au
Subscriptions:          mercury-developers-request at cs.mu.oz.au
--------------------------------------------------------------------------



More information about the developers mailing list