[edited to fix spelling]
Fishhack66 wrote:
I'm good with whatever you think makes the most sense for those (archeologists?) who will come after us.
I think fixing the labels so they are like the PDF listing makes the most sense. That means the RTSN label moves up a line (it's literally the RTS for the "Normalization" procedure and the FSUB moves up a line so it's the label that will need to be used for subtraction, and the SWPALG (swap or align) label will be brought back to where FSUB used to be. All of the references to these labels are in my section except for the following two:
Code:
FOCAL-CompletedSource_v3d_Aresco.txt Line 1063
JSR RTSN ; SUBTRACT COUNTER FROM UPPER LIMIT
becomes
JSR FSUB ; SUBTRACT COUNTER FROM UPPER LIMIT
and
FOCAL-CompletedSource_v3d_Aresco.txt Line 1107
JSR RTSN ; SUBTRACT OFF THE GROUP NUMBER
becomes
JSR FSUB ; SUBTRACT OFF THE GROUP NUMBER
Even from those tiny snippets, I think you can see that the code now makes sense with the comments. Note that this should result in the SAME OBJECT CODE as before, but the source code will be much easier for a future code archeologist to deal with.
I've finished my section and have only one other mystery concerning a LDY# that I need help with:
Attachment:
File comment: Load Y immediate in PDF source code
Screenshot_ldyimm.png [ 18.95 KiB | Viewed 6808 times ]
was typed in as (line 3261)
Code:
LDX #FHALF ;GET THE CONSTANT .5
LDY X1 ;INTO FAC1
which looks like it is missing the # on LDY, however in the listing for the new source code we are developing, it shows this:
Code:
3345 A2 96 LDX #FHALF ;GET THE CONSTANT .5
3347 A0 80 LDY X1 ;INTO FAC1
and that A0 on the LDY line looks like it is assembling as LDY immediate, even though there is no # in front of the operand. Is there supposed to be a # on the operand? I think so. The mystery to me is how the listing has the correct opcode even though I think the source code has an error.
That's the only true error (if it is an error) I found in my section. Kudos to the typists who typed this all in from the PDF listing.