Originally Posted By: jcl
Changing software behavior just to match the documentation could break existing code

But that code was already broken in the first place. smirk

Originally Posted By: jcl
therefore we do that only when there's a real reason for it

There is: As Rakscha pointed out the currently implemented behaviour is totally inconsistent with the rest of the software.

Additionally the "_startup" check does not take into account the number of arguments of the function (does not make any sense to me. If it's a startup it shouldn't have any parameters!) This makes the whole thing even more dangerous.
Code:
#include <acknex.h>

void show_startup_text(TEXT* t)
{
	if (t)
		t->flags |= SHOW;
}

This small piece of code will cause your application to crash. Even worse on my machine the crash occurs randomly.

Last edited by Uhrwerk; 02/13/13 17:40. Reason: Why was this moved to Lite-C Programming?

Always learn from history, to be sure you make the same mistakes again...