Classic bug

[Lost User]

From one of our developers who should know better:

// Validate pView pointer
if (!pView)
{
pView->KillFuncHook();
return;
}

[Freak30]

Some time ago I found a similar thing in our codebase. It looked something like:

if(table == NULL)
{
Log(GetFormatString("Table %d is NULL in funtion xyz!", table->Number));
return false;
}

Seemed like it had been there for years before fist being hit.

[Bert Mitton]

What we used to do in our dept when we found someone else's mistake would be to print it out and hang it on the wall, complete with the name of the author and the date it was found. Quickly, people began to double check their work. Sadly, we tore down that wall. :((

[Jorgen Andersson]

Surely there must be other walls in the office.

List of common misconceptions

[Lost User]

We used to have a whiteboard with a sort of bug league table, probably time to resurrect that idea.

[Bert Mitton]

Yeah, but they're not nearly as close. It loses something when it's not so easily visible.

[Lost User]

Name ans shame! Works for Paedophiles.. :)

============================== Nothing to say.