Quote:
Originally Posted by addons_zz
Hi, i have this idea about a annoying thing about the DHUD and HUD, they dont show up at the client's console, so if i could not read all the message, i lost i forever, then I added one line of code to you code to print it at console too.
If you feel the same bothering, could you update the plugin to the others?
|
Totally forgot about that. Added.
Quote:
|
Originally Posted by HamletEagle
No, just check AMXX_VERSION_NUM if lower than 1.8.3. If so include the inc file, else do nothing.
|
I actually searched for something like that, but coulnd't find anything. Added now, thank you.
Quote:
|
Originally Posted by HamletEagle
I did not read your code(so remember this, I am not talking about the modifications you made), but I can tell you it's a bad ideea to just take a default amxx plugin, edit and release it. You should instead make a pull request on github with your changes. This is because AMXX is updated from time to time and plugins are likely updated. This means that your plugin will be stuck in time and probably not compatible or missing new amxx version features. Take this as an example: https://forums.alliedmods.net/showthread.php?t=27855 |
Well, I didn't actually edit and release it. I totally rewrote it and made a brand new one with a lot of new options. I'm not familiar with what GitHub is or how it works, but I'll take a look later. Although AMXX is updated, I personally haven't seen any major changes to the default plugins so far, especially this one. Also, the thing with a plugin being stuck probably won't affect only this one, so it's more of a "global" issue. I doubt that they will implement something better than this, and if they do, I will probably update my plugin with more options.
Quote:
|
Originally Posted by addons_zz
Of course, was good too publish it here, so you and whoever will decide to accept the pull request, could see the modifications acceptance by the community. Hence, to be more precise, you could add a poll to this thread, asking if this is a AMX must default plugin. In my opinion it is.
|
All default plugins can use a refreshment. In my opinion the current codes can be written in much more better ways, and this is example of such a thing. It probably isn't the best way of coding it, but I'm sure that it's better than the default one. I'm expecting some advice for improving my code, if needed.