zx-next-dev-guide icon indicating copy to clipboard operation
zx-next-dev-guide copied to clipboard

Misleading typo - tilemap chapter (page 86) reference to "16K slot 5" instead of "16K bank 5".

Open BloodBaz opened this issue 5 months ago • 2 comments

There isa reference to "16K slot 5" instead of "16K bank 5" in the opening sentence of section 3.7.3 Memory Organization.

Its on line 74 of vctmp23008_689661.chapter-next-tilemap.716b2660.tex

I downloaded and made the code change and commited locally but I can't seem to push it back up (I'm not familiar with using git) as I'm getting an error:

Pushing branch Remote: Permission to tomaz/zx-next-dev-guide.git denied to BloodBaz. Error encountered while pushing to the remote repository: Git failed with a fatal error. unable to access 'https://github.com/tomaz/zx-next-dev-guide.git/': The requested URL returned error: 403

Failed to push to the remote repository. See the Output window for more details.

It's just a single word change that is otherwise quite confusing mixing "slot" and "bank" in this context. Thanks for an amazing book!

BloodBaz avatar Sep 12 '25 22:09 BloodBaz

You would need to fork, commit to your fork, and request a merge.

Or post a diff patch.

But your “bug report” seems clear enough that hopefully Tomas can just fix it and close this ;-)

-Dx

On Fri, 12 Sep 2025 at 23:59, Chris Walsh @.***> wrote:

BloodBaz created an issue (tomaz/zx-next-dev-guide#48) https://github.com/tomaz/zx-next-dev-guide/issues/48

There isa reference to "16K slot 5" instead of "16K bank 5" in the opening sentence of section 3.7.3 Memory Organization.

Its on line 74 of vctmp23008_689661.chapter-next-tilemap.716b2660.tex

I downloaded and made the code change and commited locally but I can't seem to push it back up (I'm not familiar with using git) as I'm getting an error:

Pushing branch Remote: Permission to tomaz/zx-next-dev-guide.git denied to BloodBaz. Error encountered while pushing to the remote repository: Git failed with a fatal error. unable to access 'https://github.com/tomaz/zx-next-dev-guide.git/': The requested URL returned error: 403

Failed to push to the remote repository. See the Output window for more details.

It's just a single word change that is otherwise quite confusing mixing "slot" and "bank" in this context. Thanks for an amazing book!

— Reply to this email directly, view it on GitHub https://github.com/tomaz/zx-next-dev-guide/issues/48, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAA4CHYMZQCUOO42KHA3D7L3SNF5FAVCNFSM6AAAAACGMJRYOCVHI2DSMVQWIX3LMV43ASLTON2WKOZTGQYTEMJSG4YDCNY . You are receiving this because you are subscribed to this thread.Message ID: @.***>

Xalior avatar Sep 12 '25 23:09 Xalior

@BloodBaz Thanks! It you can create a pull request as suggested by @Xalior that would be ideal, otherwise it's noted and I'll update (eventually 😅)

And sorry for non responsiveness, extremely busy lately...

tomaz avatar Oct 02 '25 09:10 tomaz