This is the mail archive of the xconq7@sources.redhat.com mailing list for the Xconq project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Thoughts on terrain imaging


mskala@ansuz.sooke.bc.ca wrote:

That image-rescaling code (which I've looked at a little more since
writing my last message here) seems to be a real dog's breakfast.  There
are a couple of points where it seems to be testing for error conditions
in the very next line after that code that guarantees that the error
condition can't happen, and there are other points where a comment
explaining what's about do be done is followed by a line of code that
clearly does something else.  I think those are signs it's been tweaked
and patched several times in the past, and it may be that the best thing
to do might be to simply rewrite those routines - not a project I had
contemplated as part of the map-image feature addition, but one I'm
willing to tackle if it turns out to be necessary.

Yes, parts of that code seemed a bit gnarled, __the few exposures that I've had to it. One of the reasons that I've been content to avoid it in the past....


I appreciate the fact that you are being open-minded about what sorts of preparatory work may need to be done before getting down to "just writing the damned feature". I think that that is a healthy attitude to have when dealing with the Xconq sources.

  Best regards,
    Eric


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]