z80pack icon indicating copy to clipboard operation
z80pack copied to clipboard

Implement write-protect for IMSAI memory boards

Open udo-munk opened this issue 6 years ago • 1 comments

Priority: low Difficulty: easy

The standalone BASIC interpreter write protects it's memory, so it can't be poked to death. Just set the 1KB pages in the page table to MEM_WPROT from the output to the port. See BASIC source and manual for refernce.

udo-munk avatar Jul 04 '18 18:07 udo-munk

Hi Geovens, any problem with this improvement ?

pubpub-zz avatar Nov 12 '18 20:11 pubpub-zz

I think that an optional floating button to activate drawing is a good idea. First, It's not easy to merge your change to the current master branch. It is based on your diverged branch of code which includes your previous modifications that are unrelated to the feature, thus it has been a bit too different to the current master branch. The code framework also chenged greatly when you implemented your code beside the Root class. Your zip file is not complete so that I can't build it as is. Seems that a button image is missing. I added a button image and built successfully, and I can see your idea get somewhat functional, but weird things still happen under different circumstances. A pull request with a full implementation is a better way for us. Second, and most importantly, I think that it added too much of complexity to the code. Even if could make a perfect implementation for now, the maintainness difficulty will rise too much for futher. I think that it's not worth it. Thank you for the great idea but I discourage you continuing doing it.

geovens avatar Nov 14 '18 12:11 geovens