[d3athkai]

Thanks

[Dragonshadow]

1) #pragma semicolon 1 forces strict semicolon mode, about incudling something you don't use? I dunno

2) Not really, but it makes it easier, use #pragma semicolon 1 (see 1)

3) Action: functions have to return Plugin_Handled, Plugin_Continue, or Plugin_Stop.

4) uhhh dunno

5) probably, but its minimal.

[rhelgeby]

1) Functions in include files are usually stock functions. They are not compiled if they aren't used in your code.

3) Functions tagged with Action requires some instructions on return telling it how to deal with events and actions (usually). Plugin_Handled is usually blocking the event/action and Plugin_Continue is ignoring it. Plugin_Stop is mostly used in timer callbacks to stop the timer, as far as I know.

4) In general all handles you create should be closed when done with it to prevent memory leaks. But on some stuff there are exceptions: http://wiki.alliedmods.net/Menu_API_...g_Menu_Handles

5) Less plugins do use less memory, but as Dragonshadow said, it's so small amounts you don't need to worry about that. Like a big plugin I work on, it's still just using 1-2 MB. On most other plugins you'll only save a few KB.

[Greyscale]

4)

if (action == MenuAction_End)
	{
		/* This is called after VoteEnd */
		CloseHandle(menu);
	}

(In the handler callback) is the magic block of code. Every menu that disappears from the client's screen (or is replaced by another) calls the handler with 'action' set to MenuAction_End.

So this is the only spot you need to put a CloseHandle();

Closing the handle while in use can cause instability.