public inbox for xconq7@sourceware.org
 help / color / mirror / Atom feed
From: Eric McDonald <mcdonald@phy.cmich.edu>
To: mskala@ansuz.sooke.bc.ca
Cc: xconq7@sources.redhat.com, <xconq-hackers@lists.sourceforge.net>
Subject: Re: Thoughts on terrain imaging
Date: Tue, 23 Nov 2004 22:08:00 -0000	[thread overview]
Message-ID: <Pine.LNX.4.44.0411231422280.16849-100000@leon.phy.cmich.edu> (raw)
In-Reply-To: <Pine.LNX.4.21.0411230030220.3911-200000@diamond.ansuz.sooke.bc.ca>


Hi Matthew,

  So far, I've had only a brief time to look at your proposal. I 
will give it a more in-depth glance tonight. Some of the areas you 
touched on, I have very little knowledge about, because I was 
content to let Hans deal with them.

  If you want, I will set up a branch in CVS for you sometime in 
the next few days or week. You will need to have a Sourceforge 
user account so that you can be added to the project as a 
developer. I figure this might be easier than you submitting a 
bunch of patches which people would have to apply if they wanted 
to test out your work.

  A couple of quick responses:

On Tue, 23 Nov 2004 mskala@ansuz.sooke.bc.ca wrote:

> * Add data structures to the map to store, for each cell and cell terrain
> type, pointers to image families and subimage numbers for
> "override" images.

For the per-cell case, you may want to implement new layers. I 
believe that the 'aref' and 'aset' macros in 'world.h', IIRC, 
would be worth looking at. (Layer allocation is handled elsewhere.)

> * Change interfaces so that when they go looking for a cell terrain image
> for a given cell, if there is an applicable override image at that cell,
> they use it instead.  I'm hoping to keep the amount of per-interface code
> for this as small as possible; that should be reasonably easy to do,

Yes. The informal UI API exists primarily in 'ui.h', 'kpublic.h', 
and 'ui.c'. The more code that can kept within the common API, the 
better. As I work on the SDL interface, I will probably be moving 
more code into the common API. Just yesterday, I identified 
another chunk that is common, but currently distributed amongst 
the interfaces. The hope is that, eventually, we will have a 
well-defined, formalized UI API, and will thus be able to attract 
more UI developers.

>    http://ansuz.sooke.bc.ca/temporary/maptest.g
>    http://ansuz.sooke.bc.ca/temporary/override.gif

I will look at it tonight.

Regarding the bug reports: if you're offering to fix the bugs, 
then that's great. If you would like some help fixing them, please 
add them to the project's bug tracking system on Sourceforge, and 
I will try to help you clear out some of them.

As far as GDI memory goes, this is something that Hans has been 
complaining about for quite some time. From what little reading 
that I have done, there appears to be an upper limit on the number 
of GDI handles available (and possibly the amount of GDI object 
heap available) in Win32. It varies from version to version. So, 
this is strictly a Win32 consideration, and it may only be 
applicable to the Tcl/Tk interface, because Tk does a lot of 
things behind ones back. I think that we have more control of our 
situation in SDL. And, of course, Curses is irrelevant in this 
regard.

  Thanks,
    Eric


  reply	other threads:[~2004-11-23 19:41 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-11-23 16:44 mskala
2004-11-23 22:08 ` Eric McDonald [this message]
2004-11-24  3:18   ` mskala
2004-11-24  4:25     ` Eric McDonald
2004-11-24 13:00 ` Eric McDonald
2004-11-25  2:50   ` mskala

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=Pine.LNX.4.44.0411231422280.16849-100000@leon.phy.cmich.edu \
    --to=mcdonald@phy.cmich.edu \
    --cc=mskala@ansuz.sooke.bc.ca \
    --cc=xconq-hackers@lists.sourceforge.net \
    --cc=xconq7@sources.redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).