Crash in BeginPaint running Sams Photofact index, a free VB app
Bill Medland
billmedland at mercuryspeed.com
Mon Apr 28 12:27:03 CDT 2003
On April 28, 2003 07:13 am, Dmitry Timoshkov wrote:
> "Uwe Bonnes" <bon at elektron.ikp.physik.tu-darmstadt.de> wrote:
> > Dmitry> I've tested it before I sent the patch. Win2k just returns 0
> > for Dmitry> both APIs without a crash.
> >
> > Ah, that's fine!
> >
> > Is your test perhaps integrable in our testsuite?
>
> I don't think that every trivial change to the code requires a test suite.
This is one of my pet peeves.
Wine is imperfect. (What isn't!) To me it is frequently unclear whether a
line of code is present because it is needed to emulate (possibly weird)
Microsoft behaviour or because we want our code to be better than Microsoft's
or because of brainfade on the part of the programmer or ...
That is why I like to see facts stated twice. Knowing which functions are
supposed to handle null pointers is, to me, not trivial. (For example expand
the question; is the function handling a null pointer or is it handling all
segvs as Windows strlen does?)
Putting in a test would be even better than simply commenting that the
behaviour is known for e.g. Win95; it might point out version differences and
certainly will catch when we accidentally break the code.
--
Bill Medland
ACCPAC International, Inc.
medbi01 at accpac.com
Corporate: www.accpac.com
Hosted Services: www.accpaconline.com
More information about the wine-devel
mailing list