-
Notifications
You must be signed in to change notification settings - Fork 57
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
Adding a demofilter #32
Conversation
…ing. If splitnetdemos was enabled when the autorecording was previously enabled, it properly disables it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code looks sound. I'm just leary about introducing 7 more CVARs. Perhaps there's another way to encode the filter options without so many CVARs.
The problem, is that I first planned a byte system, but it'd be absolutely messy for the player if modifying from the .cfg . Individual CVARs were far easier to manage in case of a problem though. |
@@ -216,6 +226,11 @@ value_t OnOffAuto[3] = { | |||
{ 2.0, "Auto" } | |||
}; | |||
|
|||
value_t DemoRestrictions[2] = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whys is 0 allow and 1 restrict? This is OK as far as setting in the application. This is confusing in odamex.cfg because having cl_autorecord_filter_duel set to 1 would imply that it would record but it's the opposite.
To me it seems we should flip the DemoRestrictions values and default the cvar to 1.
…ofilter # Conflicts: # client/src/m_options.cpp
Requested by the community, to be able not to record everything from every gamemode.
I also added these options under the "Network options" submenu.