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

CAEWeaponAudioEntity #753

Merged
merged 26 commits into from
Oct 7, 2024

Conversation

Pirulax
Copy link
Collaborator

@Pirulax Pirulax commented Jul 21, 2024

No description provided.

@Pirulax Pirulax requested a review from yukani July 21, 2024 17:31
@Pirulax Pirulax added the tested This PR has been tested and is ready to be merged label Jul 21, 2024
}

CAEWeaponAudioEntity* CAEWeaponAudioEntity::Constructor() {
this->CAEWeaponAudioEntity::CAEWeaponAudioEntity();
CAEWeaponAudioEntity::CAEWeaponAudioEntity();

Check warning

Code scanning / PREfast

Don't try to declare a local variable with no name (es.84). Warning

Don't try to declare a local variable with no name (es.84).
source/game_sa/Audio/AEAudioEnvironment.cpp Outdated Show resolved Hide resolved
source/game_sa/Audio/AEAudioEnvironment.cpp Outdated Show resolved Hide resolved
source/game_sa/Audio/entities/AEExplosionAudioEntity.cpp Outdated Show resolved Hide resolved
source/game_sa/Audio/managers/AESoundManager.h Outdated Show resolved Hide resolved
@Pirulax
Copy link
Collaborator Author

Pirulax commented Sep 22, 2024

Tested, it seems to work just fine.

@Pirulax Pirulax merged commit c80350b into gta-reversed:master Oct 7, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tested This PR has been tested and is ready to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants