On Mon, 8 May 2000, David Pinson wrote:

> This is to announce an implementation of Mode 12h support
> for dosemu under X11 using instruction emulation.

Good starting point, bgidemo now does something visible ;-)

Some remarks:

  - it should not only intercept/emulate writing, but also reading
    (write protecting the regen-buffers is not enough).
    The current patch makes Borland's bgidemo not display correctly
    due to this:
      - GetImage doesn't work
      - Patterns display not correct in the 'Random Bars' and
        the 'Bar / Rectangle' demo, may be others too.
        (only vertical lines seem to be generated)

  - Bgidemo freezes on the 'LineRel / MoveRel' demo
    (could be something wrong string/move instructions)

  - the 8x8 font table hack at FFA6E isn't necessary, all it needs
    is a

     extern int load_file(char *name, int foffset, char *mstart, int msize);
     load_file("/dev/mem", GFX_CHARS, (char *) GFX_CHARS, GFXCHAR_SIZE);

    instead of

     for(i=0; i<1024; ++i) *(unsigned char *)(0xFFA6E + i) = font1[i];

    Bart is right, we solved that, however only for graphics on console
    and copying the font isn't done for X.

  - there are some parts in the code, which will need some optimization
    (e.g. the 'rep' loops working directly on the emulated machine
     registers). I'm sure the code can be made faster.
    ... but this should be done _after_ the code is working correctly.

  - The patch also should handle other than mode12 16-color modes
    of different resolutions. It can do that.


I've also to 'answer' on the implicit complain in the README (;-):

   "I just got tired of waiting for mode 12h X support and did it for my
    own use."

Well, one needs not to wait for 'someone' in a developer team to fulfill
personal wishes;-) One simply can go forward and implement it.
This is the normal way OpenSource development works. Its not enough to
specify wishes on a project's mailing list, it also needs someone to do
the work. Hence, doing the here discussed patch and giving it to the
comunity was the right thing. I cannot see any reasonable other way.

I very appreciate if people could 'finetune' the patch, its a good step in
right direction.

Hans
<[EMAIL PROTECTED]>



Reply via email to