[Seta00]

Originally Posted by Greyscale I can see how this is a tough problem to solve, though. Because sometimes I need to tag each enum member with a specific data type. The question is how can we tell the compiler what we want? The tag of the member or of the array.

I found the AMXx thread about this, look at this:

Originally Posted by joaquimandrade I think it's interesting to understand the problem here. And it is: when you are using a value defined as part of an enum to index an array, the compiler fails to detect the tag of the cells of the array. Like: PHP Code: enum _:Coordinate {     X,Y,Z } new Float:coordinates[Coordinate]; public plugin_init() {     new Float:value          // Tag mismatch     value = coordinates[X]          // OK     value = Float:coordinates[X]     // OK but is stupid     value = coordinates[0 + X] }

[Fyren]

It is intentional that when using an enum value to index an array, the expression inherits the enum value's tag.

[Seta00]

Originally Posted by Fyren It is intentional that when using an enum value to index an array, the expression inherits the enum value's tag.

Yeah, that's how the enum hack works...

@Greyscale
You'll have to tag every enumeration, like this:

enum SDKLibCalls
{
    #if defined PROJECT_GAME_CSS
    Handle:SDKLibCall_TerminateRound,      /** Terminate the round with a given reason. */
    Handle:SDKLibCall_CSWeaponDrop,        /** Force client to drop weapon. */
    Handle:SDKLibCall_FlashlightIsOn,      /** Check is a client's flashlight is on. */
    Handle:SDKLibCall_FlashlightTurnOn,    /** Turn a client's flashlight on. */
    Handle:SDKLibCall_FlashlightTurnOff,   /** Turn a client's flashlight off. */
    #endif
}

[Fyren]

I'm just replying because you called it a "design flaw" when it is the intentional behavior.

[Seta00]

Originally Posted by Fyren I'm just replying because you called it a "design flaw" when it is the intentional behavior.

It makes sense, else Pawn's "structure" would be impossible to make, but in cases like this, where the enum is used just to name integer indexes it's hard to see why it works that way.

By the way, an alternative solution would be removing the name of the enum, like this:

enum
{
    #if defined PROJECT_GAME_CSS
    SDKLibCall_TerminateRound,      /** Terminate the round with a given reason. */
    SDKLibCall_CSWeaponDrop,        /** Force client to drop weapon. */
    SDKLibCall_FlashlightIsOn,      /** Check is a client's flashlight is on. */
    SDKLibCall_FlashlightTurnOn,    /** Turn a client's flashlight on. */
    SDKLibCall_FlashlightTurnOff,   /** Turn a client's flashlight off. */
    #endif
    SDKLibCalls
}

[Greyscale]

I use that enum for multiple arrays.

/**
 * The name of the sdkcall.  This is the name that should be used in the gamedata file.
 */
stock String:g_strSDKCallName[SDKLibCalls][] =
{
    "Placeholder"
    #if defined PROJECT_GAME_CSS
   ,"TerminateRound",
    "CSWeaponDrop",
    "FlashlightIsOn",
    "FlashlightTurnOn",
    "FlashlightTurnOff"
    #endif
};

Which works fine, without warnings, by the way.

I think the best solution is just to add whatever tag I need before the array name.

Handle:g_hSDKCalls[SDKCallLib_TerminateRound];

[Seta00]

Originally Posted by Greyscale I use that enum for multiple arrays. /** * The name of the sdkcall. This is the name that should be used in the gamedata file. */ stock String:g_strSDKCallName[SDKLibCalls][] = { "Placeholder" #if defined PROJECT_GAME_CSS ,"TerminateRound", "CSWeaponDrop", "FlashlightIsOn", "FlashlightTurnOn", "FlashlightTurnOff" #endif }; Which works fine, without warnings, by the way.

In this case, you're accessing the data on the second dimension, and there's only one type.

I think the best solution is just to add whatever tag I need before the array name. Handle:g_hSDKCalls[SDKCallLib_TerminateRound];

I'd remove the enum name, anyway it's your code, you decide what to do.

[Greyscale]

How am I going to index an array with an enum with no name?

[Seta00]

PHP Code:

/**
 * List of available sdktools functions.
 */
enum
{
    #if defined PROJECT_GAME_CSS
    SDKLibCall_TerminateRound,      /** Terminate the round with a given reason. */
    SDKLibCall_CSWeaponDrop,        /** Force client to drop weapon. */
    SDKLibCall_FlashlightIsOn,      /** Check is a client's flashlight is on. */
    SDKLibCall_FlashlightTurnOn,    /** Turn a client's flashlight on. */
    SDKLibCall_FlashlightTurnOff,   /** Turn a client's flashlight off. */
    #endif
    SDKLibCalls
}

/**
 * Handles to all the pre-defined sdkcalls.
 */
new Handle:g_hSDKCall[SDKLibCalls];

/**
 * Terminates the round. (CS:S)
 * 
 * @param delay     Time to wait before new round starts.
 * @param reason    Round end reason.  See enum RoundEndReasons.
 */
stock SDKToolsLib_TerminateRound(Float:delay, RoundEndReasons:reason)
{
    SDKCall(g_hSDKCall[SDKLibCall_TerminateRound], delay, _:reason);
}

/**
 * Forces client to drop a weapon. (CS:S)
 * 
 * @param client    The client index.
 * @param weapon    The weapon entity index to force client to drop.
 */
stock SDKToolsLib_CSWeaponDrop(client, weapon)
{
    SDKCall(g_hSDKCall[SDKLibCall_CSWeaponDrop], client, weapon, true, false);
}