Skip to content

EFFWave Effect#436

Merged
Gunoshozo merged 2 commits intomasterfrom
EFFWave
Apr 3, 2026
Merged

EFFWave Effect#436
Gunoshozo merged 2 commits intomasterfrom
EFFWave

Conversation

@Gunoshozo
Copy link
Copy Markdown
Contributor

@Gunoshozo Gunoshozo commented Mar 24, 2026

EFFWave Effect

  • Extend DrawPrimitives method to swap current frameBuffer
  • Reorder some stuff in Wave methods, to make less unnecessary calculations
  • Add default log to the InstBGeffectWave, to catch newer versions of instruction

Rendering Hot Path Optimizations

  • Character2D::Render Moved out a loop invariant
  • Renderer::InsertVertices unite 2 passes over the same vector + fix possible side effect of using restart indices
  • Renderer::DrawPrimitives preallocate TransformedVertices and use spans of it + branch on sheet.IsScreenCap to std::transform or std::copy

@Gunoshozo Gunoshozo self-assigned this Mar 24, 2026
@Gunoshozo Gunoshozo linked an issue Mar 24, 2026 that may be closed by this pull request
@Gunoshozo Gunoshozo force-pushed the EFFWave branch 5 times, most recently from 8467787 to f409fbe Compare March 25, 2026 16:24
@Gunoshozo Gunoshozo marked this pull request as ready for review March 25, 2026 16:40
@Gunoshozo Gunoshozo marked this pull request as draft March 25, 2026 18:18
@Gunoshozo Gunoshozo force-pushed the EFFWave branch 5 times, most recently from ec046ec to edfcd7f Compare March 29, 2026 09:29
@Gunoshozo Gunoshozo marked this pull request as ready for review March 29, 2026 18:40
- Reorder some stuff in Wave methods, to make less unnecessary calculations
- Add minimal support for alpha ScrWork of wave effect from newer versions of an engine
- Add default log to the InstBGeffectWave, to catch newer versions of instruction
- Character2D::Render Moved out a loop invariant
- Renderer::InsertVertices unite 2 passes over the same vector + fix possible side effect of using restart indices
- Renderer::DrawPrimitives preallocate TransformedVertices and use spans of it + branch on sheet.IsScreenCap to std::transform or std::copy
@Gunoshozo Gunoshozo merged commit 58e4717 into master Apr 3, 2026
11 checks passed
@Gunoshozo Gunoshozo deleted the EFFWave branch April 3, 2026 20:21
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.

EFFWave Implementation

3 participants