Skip to content

Conversation

@eezstreet
Copy link

After this commit, all of the DOOM 3 base code and Resurrection of Evil SDK code exist as one unified codebase, producing two DLLs but sharing 90% of the same code. Anything that is different between them (should) be #ifdef'd out.

The primary motivation for doing this is to eliminate code reuse. I want to develop a mod for both Resurrection of Evil and the base game, but I don't want to make my changes twice. Thus, having a unified codebase like this makes a lot of sense.

eezstreet added 2 commits May 24, 2023 02:05
After this commit, all of the DOOM 3 base code and Resurrection of Evil
SDK code exist as one unified codebase, producing two DLLs but sharing
90% of the same code. Anything that is different between them (should)
be `#ifdef`'d out.

The primary motivation for doing this is to eliminate code reuse. I want
to develop a mod for both Resurrection of Evil and the base game, but I
don't want to make my changes twice. Thus, having a unified codebase
like this makes a lot of sense.
@eezstreet
Copy link
Author

Particle effects are currently not moving. Will need to fix that before this gets merged in.

@eezstreet
Copy link
Author

This should be ready to merge whenever someone has the chance to review it.

@FriskTheFallenHuman
Copy link

I've checked the code myself but there's one weird thing i have notice with stock and merged codebase:

This is Mars City 2 map loaded throught console:
image

And this is the same map on stock pre merge codebase:
image

It seems like in vanilla the game its supposed to give you the shotgun but you never get the shotgun at all

other than that it seems solid

@FriskTheFallenHuman
Copy link

Enpro Plant has the same issue has Mars City 2

Merge Code:
image

Stock:
image

@FriskTheFallenHuman
Copy link

Seems like something is off with the player inventory / weapons pickup after the merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants