[PATCH 1/3] lz32/tests: Test more last errors set by
LZOpenFile[AW].
Saulius Krasuckas
saulius2 at ar.fi.lt
Tue Oct 3 12:40:33 CDT 2006
Hello, Dmitry,
* On Wed, 27 Sep 2006, Dmitry Timoshkov wrote:
> * "Saulius Krasuckas" <saulius.krasuckas at ieee.org> wrote:
> > memset(&filled_0xA5, 0xA5, OFS_MAXPATHNAME);
> > memset(&test, 0xA5, sizeof(test));
> > full_file_path_name_in_a_CWD(filename_, expected, FALSE);
> > + SetLastError(0xfaceabee);
>
> Please use a usual value we all agreed to use: 0xdeadbeef.
I remember only countless proposals by you to agree on it, but no any
documented place marking such agreement. If that's of such an important
matter, why still no one has added some janitorial page about it? (in the
wiki for example)
> It makes sense to check last error value only if an API has failed,
> while you are adding last error checks everywhere. Unless you have an
> application that depends on it, that's wrong.
Well, thanks. I'll change that, but is it wrong just because it slightly
increases code complexity for no direct benefit? I just want to know for
future.
More information about the wine-devel
mailing list