[Rul4]

Originally Posted by NiQu Did you think of people that is using no-flash also could use ESP with that said i wanna say this plugin is to no use Make it block ESP or something and it would be great.

Stripping player's information should block ESP or wallhacks too during the period you are flashed. It will not block soundhacks though, because they get their info through an entirely different way.

The only purpose for this plugin is removing advantage during "blind" time. It should work well toghether with other anticheats, such as anti-wallhacks.

[wrecked_]

Originally Posted by Rul4 Please tell me what's on your mind. It is always good learning something else.

Hmmmm, well from what I can see:

• Remove the amxmisc include
• #include <cstrike>
  • Use cs_get_user_team instead of get_user_team (sometimes returns false results)
  • Remove those team defines

But that's just me. I'm sure Hawk has more significant suggestions.

[Hawk552]

Originally Posted by wrecked_ Remove the amxmisc include

Don't do that.

Originally Posted by wrecked_ Use cs_get_user_team instead of get_user_team (sometimes returns false results) -> Remove those team defines

Do that.

PHP Code:

    register_cvar("anti_noflash_version", VERSION, FCVAR_SPONLY | FCVAR_SERVER); 


FCVAR_SPONLY is meaningless for any multiplayer game. All it does is stop clients from being able to change it, which they couldn't do in the first place.

PHP Code:

 (!id) || !is_user_connected(id) 


Unnecessary brackets on the !id check, and it's also probably useless to begin with. I suggest you just check if id is between 1 and 32 and skip the connected check, or just remove the !id check and leave is_user_connected().

PHP Code:

            new Classname[33];
            pev(ent, pev_classname, Classname,32); 


Make this static.

PHP Code:

        remove_flash(id) 


Inconsistent use of semicolons. Use #pragma semicolon 1 (then remove it before release) to see if you missed any.

PHP Code:

public remove_flash(id){
        
    //Remove flashing form target player for testing purposes
    message_begin(MSG_ONE, g_msg_screen_fade, {0, 0, 0}, id);
    write_short(0);
    write_short(0);
    write_short(0);
    write_byte(0);
    write_byte(0);
    write_byte(0);
    write_byte(0);
    message_end();    

    } 


Inconsistent indentation.

PHP Code:

message_begin(MSG_ONE, g_msg_screen_fade, {0, 0, 0}, id); 


Use MSG_ONE_UNRELIABLE.

[Rul4]

I've changed the code according to your suggestions.

Thanks to all of you.

[1337]

L 04/24/2010 - 04:25:23: [CSTRIKE] Invalid player 1 L 04/24/2010 - 04:25:23: [AMXX] Run time error 10 (plugin "antinoflash.amxx") (n ative "cs_get_user_team") - debug not enabled! L 04/24/2010 - 04:25:23: [AMXX] To enable debug mode, add " debug" after the plug in name in plugins.ini (without quotes).

L 04/24/2010 - 047:13: [CSTRIKE] Invalid player 1 L 04/24/2010 - 047:13: [AMXX] Displaying debug trace (plugin "antinoflash.amxx ") L 04/24/2010 - 047:13: [AMXX] Run time error 10: native error (native "cs_get_ user_team") L 04/24/2010 - 047:13: [AMXX] [0] antinoflash.sma::fw_PlayerSpawn (line 143 )

mb fix it man?

[Rul4]

Originally Posted by 1337 mb fix it man?

Which mod are you running? CS or another one?

[lazarev]

And how can I detect if player is using noflash? =\

[Rul4]

Originally Posted by lazarev And how can I detect if player is using noflash? =\

You can't. At least from the server's perspective. That is why I made this plugin.

[Duffnexter]

L 04/28/2010 - 211:41: [CSTRIKE] Invalid player 1 L 04/28/2010 - 211:41: [AMXX] Run time error 10 (plugin "antinoflash.amxx") (native "cs_get_user_team") - debug not enabled! L 04/28/2010 - 211:41: [AMXX] To enable debug mode, add "debug" after the plugin name in plugins.ini (without quotes).

any ideas how to fix it?

[1337]

Originally Posted by Rul4 Which mod are you running? CS or another one?

CS
im not alone who get this bug...
im sure EVERYONE gets this bug