[Bara]

Originally Posted by Dr. Api Ok what will should do instead ?

You cna use old syntax.

[Dr. Api]

Originally Posted by Bara You cna use old syntax.

I prefer to use one syntax !

[Dr. Api]

Tag mismatch when I use PrintToDev

PHP Code:

/***********************************************************/
/****************** GET CLIENT BY STEAM ID *****************/
/***********************************************************/
stock void GetClientBySteamIDEx(char[] steamid)
{
    char steamId[64];
    for(int i=1;i<=MaxClients;++i)
    {
        if(!Client_IsIngame(i))
            continue;
        if(!IsClientAuthorized(i))
            continue;
        GetClientAuthId(i, AuthId_Steam2, steamId, sizeof(steamId));
        if (StrEqual(steamId, steamid))
            return i;
    }
    return false;
}

/***********************************************************/
/*********************** PRINT TO DEV **********************/
/***********************************************************/
stock void PrintToDev(bool status, char[] format, any:...)
{
    if(status)
    {
        int client = GetClientBySteamIDEx(DEVELOPER);
        if(client)
        {
            char msg[MSG_LENGTH];
            char msg2[MSG_LENGTH];
            Format(msg, MSG_LENGTH, "%s%s%s%s", CHAT_HIGHLIGHT_INC, TAG_CHAT_INC, CHAT_NORMAL_INC, format);
            VFormat(msg2, MSG_LENGTH, msg, 3);
            
            Handle hBf;
            hBf = StartMessageOne("SayText2", client);
            if (hBf != INVALID_HANDLE)
            {
                if (GetUserMessageType() == UM_Protobuf)
                {
                    PbSetInt(hBf, "ent_idx", client);
                    PbSetBool(hBf, "chat", false);

                    PbSetString(hBf, "msg_name", msg2);
                    PbAddString(hBf, "params", "");
                    PbAddString(hBf, "params", "");
                    PbAddString(hBf, "params", "");
                    PbAddString(hBf, "params", "");
                }
                else
                {
                    BfWriteByte(hBf, client); 
                    BfWriteByte(hBf, 0); 
                    BfWriteString(hBf, msg2);
                }
                EndMessage();
            }
        }
    }
} 


[Bara]

"any:..." -> "any ..."

[Dr. Api]

this is not the problem actually.

[_AeonOne_]

Originally Posted by Dr. Api this is not the problem actually.

GetClientBySteamIDEx has the wrong return type. It's void, but should be int.
Additionally, returning false to an int is bad coding style (in my opinion) - return 0!

[Dr. Api]

Originally Posted by _AeonOne_ GetClientBySteamIDEx has the wrong return type. It's void, but should be int. Additionally, returning false to an int is bad coding style (in my opinion) - return 0!

It was return 0 first, I changed when I tried something else. Bu thx I miss stock int instead on stock void.

[nergal]

Just asking, what are the leftover plans/updates for methodmaps? I'd like to start utilizing them and would like to know about any future changes so that I can quickly fix them up.

[BAILOPAN]

I think methodmaps are feature complete, unless we've missed something.

[kossolax]

We may also consider an automatic porting tool for plugins once enough of the API is available.

It might be interesting to have a link of the new syntax to every .sp attached on the forum.

I mean from:

PHP Code:

 Get Plugin or Get Source 


To:

PHP Code:

Get Plugin, Get SM16 Source, Get SM17 Source 


or something like that.

So more example will be available to everyone, and more coders will switch to the new syntax.