[Firippu]

Required Map

1. surf_simpsons_final

Description

1. This plugin prevents shortcuts from being used, which makes finishing the map more difficult.
2. Additionally, when a player uses the finished button at the end of the map it will announce with their name that they finished. Preview: http://imgur.com/sx3rlOK

Notes

1. Once a player uses the finished button it will not display the name of that player anymore until they spawn again. This is to avoid hudmessage abuse. For example, a player staying in the bar pressing the finished button every minute displaying their name.

installation - This plugin is designed to use the map specific plugin method.

1. Create a new file with shown structure and naming
   PHP Code:

   addons/amxmodx/configs/maps/plugins-surf_simpsons_pro.ini 

2. Write the plugin file name within the newly created .ini file as shown and save it
   PHP Code:

   surf_simpsons_pro.amxx 


[soolt]

good job but sad that only for one map.

[#8 SickneSS]

lol nice

[Hawk552]

There are a few mistakes and optimizations that can be made, however it is acceptable.

If you would like any information regarding possible adjustments you could make or things you could do to make this better, please feel free to post here or PM me.

Approved.

[Hawk552]

Changes have been requested via PM.

For starters, inconsistent use of semicolons.

PHP Code:

new const MDL[MaxMDL][0] = 


I'm surprised this works. You should use [] or a constant that will store all of the strings properly.

PHP Code:

    new mapname[32]
    get_mapname(mapname,31)
    
    if(equali(mapname,"surf_simpsons_final")) 


Unnecessarily long mapname variable. Shorten it to 20 characters and then pass 19 as the max length for get_mapname().

PHP Code:

new mdl; 


Avoid creating variables in loops.

PHP Code:

    return FMRES_IGNORED; 


Meaningless return; this isn't even an FM forward.

[fezh]

Originally Posted by Hawk552 PHP Code:     return FMRES_IGNORED;  Meaningless return; this isn't even an FM forward.

Originally Posted by fakemeta_const #define FMRES_IGNORED 1 // Calls target function, returns normal value

Originally Posted by ham_const #define HAM_IGNORED 1 /**< Calls target function, returns normal value */

They are equal return values.. I know it's for readability, but I can't see te point of marking it :/