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.

Sign up for GitHub

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

Jump to bottom

Misc raycaster opts #5

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Misc raycaster opts #5

wants to merge 6 commits into from

Conversation

Daft-Freak
Copy link
Collaborator

Was scrolling through here doing something else and got distracted. ~26 -> 19ms on the thing I was testing.

@Daft-Freak
Simplify an integer modulo
09fe1a2 
Saw this when scrolling past, bit too much floating point going on here...
@Daft-Freak
Use some v_span in raycaster
b960617 
Saves a couple of ms
@Daft-Freak
Avoid clamping sprite distance then not use the clamped value
c65d668
@Daft-Freak
Optimise raycaster floor drawing a bit
48bbef1 
Modify the colour instead of going through screen.pixel twice and blending
@Daft-Freak
Skip initial clear in raycaster
1785bff 
We draw the entire screen anyway
@Daft-Freak
Flip raycaster sky rendering around
ba77df8 
A little faster to do the fill in one go
@Daft-Freak
Copy link
Collaborator Author

Pretty sure internal-api.service.iad.github.net being inaccessible is not my problem...

@Gadgetoid
Copy link
Contributor

Were you poking at this? -

raycaster/raycaster.cpp:27:2: warning: suggest braces around initialization of subobject [-Wmissing-braces]
        Vec2(0,0),
        ^~~~~~~~~~
        {

@Daft-Freak
Copy link
Collaborator Author

Hmm, nope (I'm not getting that warning anyway). Think I was here for the big map rework.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Daft-Freak @Gadgetoid