Skip to content

simple shadow :)#746

Open
spongeis5 wants to merge 3 commits into
bfbbdecomp:mainfrom
spongeis5:main
Open

simple shadow :)#746
spongeis5 wants to merge 3 commits into
bfbbdecomp:mainfrom
spongeis5:main

Conversation

@spongeis5
Copy link
Copy Markdown

PLEASE oh PLEASE review my code i dont wanna screw anything up 😔

Copy link
Copy Markdown
Member

@SquareMan SquareMan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this and welcome to the project! I haven't fully reviewed the function code yet as there are some major structural issues introduced by these struct edits you've made. Feel free to ask for more help in the discord if you need it but once this PR passes CI one of us will give it a more thorough review.

Comment thread src/SB/Core/x/xGlobals.h Outdated
Comment thread src/SB/Core/x/xShadowSimple.h Outdated
Comment thread src/SB/Core/x/xShadowSimple.cpp Outdated
@spongeis5
Copy link
Copy Markdown
Author

thank you so much for the feedback !!

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 8, 2026

main/SB/Core/x/xShadowSimple

Section From To Bytes
.bss 99.69% 100.00% +44
.rodata 14.29% 100.00% +13
📈 .sdata2 50.00% 77.27% +13
📈 .text 14.98% 96.07% +4093
Function From To Bytes
📈 shadowRayEntCB(xEnt*, void*) 0.00% 94.18% +794
📈 xShadowSimple_SceneCollide(xShadowSimpleCache*, xVec3*, float) 0.00% 97.70% +484
📈 xShadowSimple_CalcCorners(xShadowSimpleCache*, xEnt*, float, float) 0.00% 98.35% +476
📈 xShadowSimple_AddVerts(xShadowSimpleCache*) 0.00% 89.67% +326
📈 xShadowSimple_Init() 40.35% 92.94% +172
📈 xShadowSimple_CacheInit(xShadowSimpleCache*, xEnt*, unsigned char) 56.84% 98.75% +147
📈 xShadowSimple_Add(xShadowSimpleCache*, xEnt*, float, float) 0.00% 94.52% +1111
xShadowSimpleQueue::operator=(const xShadowSimpleQueue&) 0.00% 100.00% +44
xShadowSimple_Render() 0.00% 100.00% +536

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.

2 participants