thejayps

Results 25 issues of thejayps

Several recent proc.reviews have taken place where the [.source-approved](https://github.com/Ravenbrook/mps/blob/586336a7021c2276a3f798ad10e6e4ab5f438226/procedure/entry.universal.rst) criteria has failed, since there isn't obviously any current workflow convention or procedure that funnels any source material (e.g emails, git...

process
optional

A user of the MPS has reported performance issues when the MPS is used to manage large heaps. Problems are reported to start at a heap size of around 6GB...

essential

See https://github.com/Ravenbrook/mps/pull/166#issuecomment-1773701151

process
essential

"Read the Documentation" does not point to the current documentation hosted at readthedocs "Fork it on Github" presupposes that a visitor intends to fork the MPS. They might want to...

git-migration
essential

We should probably diligently log every message we send to the mutator, so we know exactly what information the MPS gave the mutator when we're trying to debug something. We...

optional

https://github.com/Ravenbrook/mps/blob/master/manual/source/release.rst This concerns the release of 1.118.0 and is strongly related to #110 In https://github.com/Ravenbrook/mps/blob/master/manual/source/release.rst (perma: https://github.com/Ravenbrook/mps/blob/554513b16461ec0bbf82856dfe6b0528518c3658/manual/source/release.rst) the mps_addr_object function (item 8 in "new features") is described as "new" whereas...

nice

ztfm.c includes a test whose intention is to verify that a transform runs to completion in the absence of ambiguous references to objects in the transform. The test setup isn't...

essential

Transforms test in ztfm.c (at the time of writing, in branch #214) lacks clear comments for most of the code, contains unused code, and contains a somewhat overengineered parser (which...

essential

https://github.com/Ravenbrook/mps/pull/223/files#r1197864295 The above comment was noted during review of mps_addr_object, with @rptb1 commenting verbally during the review that the main reason for lack of clarity was the undefined use of...

optional
review-raise

There are several non-straightforward conventions to do with writing and organising tests. Most of these conventions seem to be from the early pre-Ravenbrook years of the MPS: - There are...

documentation
optional