-
Notifications
You must be signed in to change notification settings - Fork 179
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New port: DOSVGA #84
base: master
Are you sure you want to change the base?
New port: DOSVGA #84
Conversation
Reduces the use of the plane register, which speeds up the write when running under emulation.
new_mode might be used uninitialized selected_size initialized wrong for rows=0 and cols=0
Possibly enough for Borland also, but this is not yet tested.
Only dosvga can compile as 16 bit and also show full color, and so this bug has gone unnoticed.
This is disabled in 4 bit color mode, because it doesn't seem to work, and is not relevant in direct color modes.
Also, fix incorrect saving of bytes under cursor in 15 and 16-bit modes.
Just compiled it, using Watcom 1.97 (haven't tried other compilers as yet), and it runs Just Fine in DosBox on my Xubuntu system. (Very slowly for a 16-bit compile, but at surprisingly decent speed in the much more reasonable 32-bit flat memory model.) |
Thank you, Bill. |
That does help. |
OK, just tried this out with my fork and, whaddaya know, it works nicely with only a few modifications. |
Guess I gotta try it out. |
Um. You have a point about the licence issue. |
I should perhaps note here that I've switched from GNU Unifont to a public domain VGA font. So we ought to be legal again. The font can be 8xheight where height <= 16, and you can now specify a font with the |
Also, draw cursor in reverse video using the foreground color.
I should have mentioned in the commit for the PSF fonts: this also provides PDC_WIDE support. |
Also, bring in @Bill-Gray's font.h; the ROM font doesn't seem to work in some configurations
I'll hold off on integrating these (very nice!) changes in my fork, both because I'm a little short on time and because it probably makes sense for me to wait until you're "done", rather than try to bring bits in piecemeal. |
@Bill-Gray I'm confused now, isn't the dosvga port in your repo using the most current code found here? If yes are there some other things than the 4.x feature set that you've adjusted? |
True. Back on March 3, there had been quite the flurry of commits for the DOSVGA port, and I thought I'd hold off on pursuing a moving target. It's been stable since then, so on May 1, I pulled the rest of @chasonr's changes (use of PSF2 fonts and wide-char builds) into my fork. The DOSVGA in my fork is now identical to @chasonr's, except for about a dozen lines. |
Is there any option for wide support in this fork? https://github.com/Bill-Gray/PDCurses/pull/136 hints that as soon as wide is defined a bunch of type mismatch errors and truncation warnings occur. |
This port compiles on MS-DOS, using the DJGPP, Watcom (16 or 32 bit) or Borland (16 bit only) compilers. It expects to find a VGA, and switches it to graphical mode to display the text.
If it finds VESA BIOS Extensions, it can use them to produce a larger display. The font size is fixed at 8 by 16 pixels.
It supports A_UNDERLINE, A_LEFT and A_RIGHT. Blink is mapped to bright background.
A correction to the testcurs.c demo is included, to fix an integer overflow in the extended color test.