Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TTT: Remove unused things #1226

Merged
merged 1 commit into from Aug 7, 2016
Merged

TTT: Remove unused things #1226

merged 1 commit into from Aug 7, 2016

Conversation

markusmarkusz
Copy link
Contributor

I hate GitHub.
I cannot reopen my old pull request.

So finally I'll never touch this hook (TTTDelayRoundStartForVote) again.
@Bo98 found some (in my opinion) very crazy coded mapvotes.
They use this hook. So indeed it seems like that there are people who use this hook.

About mapcycle:
I was a bit iritated because in the ttt home page it says:

ttt_always_use_mapcycle (def. 0):
Enables the use of your mapcycle.txt even if voting is enabled. Since GMod 13 no voting exists (at the time of writing), so this does nothing.

And in mapcycle.txt it says:

It is empty because Garry's Mod does not use mapcycle.txt
It only exists so that mapcycle.txt from mountable games ( Portal 2 ) does not load and spam the console with errors

So I thought in isn't really used and I didn't thought about this.
But game.GetMapNext() checks mapcycle.txt
So mapcycle is used in TTT.

Anyway these changes can be made:

  • Removing this if-query in cl_voice.lua
  • Removing "ttt_announce_deaths"-ConVar
  • Removing "ttt_always_use_mapcycle"-ConVar

These things are really not used.
(My old PRs: #1225 and #1223 )

Next time I will take more time for chaning something. So I can test it and search the internet about it.
I apologize if I confused someone. :(

@robotboy655 robotboy655 added the TTT The pull request is for TTT and will be handled by svdm. label Aug 5, 2016
@svdm svdm merged commit 54aeb7e into Facepunch:master Aug 7, 2016
@markusmarkusz markusmarkusz deleted the patch-5 branch September 10, 2016 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TTT The pull request is for TTT and will be handled by svdm.
Projects
None yet
3 participants