Hello, Dmitry,
* On Wed, 27 Sep 2006, Dmitry Timoshkov wrote:
- "Saulius Krasuckas" saulius.krasuckas@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.