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

Changes in renderer2, addition of SSE and other improvements #2570

Merged
merged 160 commits into from
May 20, 2024

Conversation

rafal1137
Copy link
Member

This is about around 160 commits. Some of them does not follow naming scheme.
Current status:
Something went wrong during porting changes from original corec branch.
It fails in tr_animation_mdm.c
Added more SSE2 Instricts to get better perfomance. This is Athlon 64+, Pentrium 4 Prescot and up
SSE and renderer2 is off by default since its needs to be tested on other platforms too.

q_math.h and rest of the renderer2 .c and .h files not included
This may result in memory corruption due to win headers being repacked in another way
rafal1137 and others added 25 commits May 11, 2024 16:51
This was driving me nuts
* renderer: remove unused functions

* cgame: remove unused macros & variables

* cgame: fix unambiguous usage of && and || in conditional statement

* qcommon: exclude FS_AddPackToPath from dedicated server

* qcommon: ensure packName doesn't point to null terminator

This shouldn't ever happen anyway as you can't have an empty filename, but this would blow up if it somehow happened.

* game: remove unused variable

* server: remove unused variables

* tvgame: remove unused variables

Should clean up the entire LUA code from tvgame really as it's not used there.

* qcommon: remove unused function

* qcommon: replace deprecate cURL function

CURLOPT_PROGRESSDATA has been deprecated in favor of CURLOPT_XFERINFOFUNCTION, the functionality remains the same however.

* omni-bot: fix incorrect memset on non-trivial datatype

* client: fix redefinition of CMD_BACKUP, refs #2494

Flip-flop CMD_BACKUP/MASK_VET to CMD_BACKUP/MASK and add CMD_BACKUP/MASK_ETL with the increased size in order to avoid redefinition of CMD_BACKUP in server.h
* Provide simple helper for cJSON location

If BUNDLED_LIBS is disabled and cjson library does not ship its cmake
config for some reason, attempt to find it the other way. That is the
case on current Fedora cjson package.

Provide CJSON_LIBRARY in addition to CJSON_LIBRARIES variable.

* Remove compat CJSON_LIBRARY variable usage

Use just CJSON_LIBRARIES similar to other modules.

* Try cJSON config-only search as the first step

Use config by default even when FindcJSON module is preferred. Try
CONFIG only and if that works, not not attempt other methods. Should
behave exactly the same way as if FindcJSON module were not present.
CURLOPT_XFERINFOFUNCTION takes curl_off_t (long) as args instead of double
…quest, fixes #2256

When "sv_hidden" is enabled now, the challenge value is checked to be a
valid active challenge, and if so, then the info request is responded to.
Copy link

sonarcloud bot commented May 20, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
2 Security Hotspots
5.4% Duplication on New Code (required ≤ 3%)
E Reliability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

@rafal1137 rafal1137 merged commit a21f1f4 into master May 20, 2024
8 of 9 checks passed
Vorschreibung added a commit to Vorschreibung/etlegacy that referenced this pull request May 22, 2024
Vorschreibung added a commit to Vorschreibung/etlegacy that referenced this pull request May 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants