Code changes

Chat about anything CX16 related that doesn't fit elsewhere
Post Reply
paxdiablo
Posts: 1
Joined: Mon May 23, 2022 6:20 am

Code changes

Post by paxdiablo »


Just wondering (to the authors) whether you're amenable to (non-functional) changes to the code base such as adding comments to the ROM code (or even functional changes if it can be improved). I thought I'd have a look-see but soon realised that the actual commentary within many files is non-existent.

kelli217
Posts: 535
Joined: Sun Jul 05, 2020 11:27 pm

Code changes

Post by kelli217 »


The functional changes are always being tweaked. It's why we're on ROM version 41. As far as comments go, Michael probably started with copious comments but he's been over it so much by now he could probably recite any given KERNAL or BASIC function's code from memory.

Wavicle
Posts: 285
Joined: Sun Feb 21, 2021 2:40 am

Code changes

Post by Wavicle »



On 5/23/2022 at 12:21 AM, paxdiablo said:




Just wondering (to the authors) whether you're amenable to (non-functional) changes to the code base such as adding comments to the ROM code (or even functional changes if it can be improved). I thought I'd have a look-see but soon realised that the actual commentary within many files is non-existent.



The decision is ultimately up to @Michael Steil. He's an approachable and reasonable person though, so if your changes correctly document the code and focus on documenting functions that need it (as opposed to every single line that is sometimes seen in assembly code), I think they would have a good chance of getting merged. Michael's favorite response anytime anybody suggests fixing something in the ROM is: "Pull request?"

image.thumb.png.f108b3c1d679712b70919c8f9965c307.png

ZeroByte
Posts: 714
Joined: Wed Feb 10, 2021 2:40 pm

Code changes

Post by ZeroByte »


Here are 77 ideas. LOL

Post Reply