omar/panopainter
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Promote next stroke dispatch task

Browse Source
This commit is contained in:
omigamedev
2026-06-13 16:56:27 +02:00
parent 0aa0ac4497
commit f3364a96ae
1 changed files with 20 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

29
docs/modernization/tasks.md
View File

@@ -509,6 +509,10 @@ Done Checks:
Progress Notes: Progress Notes:
- 2026-06-13: `STR-004` is now the next ready slice for the last inline stroke
dispatch glue; `LATER-003` is effectively at the binding-only tail. Keep
this task focused on removing or consolidating helper-builder wiring rather
than reopening the landed execution helpers.
- 2026-06-13: `pp_paint_renderer_stroke_execution_tests` now also covers the - 2026-06-13: `pp_paint_renderer_stroke_execution_tests` now also covers the
new retained main-pass texture dispatch helper builder and its pattern/mixer new retained main-pass texture dispatch helper builder and its pattern/mixer
wiring. Next slice should target another narrow `stroke_draw()` seam or stop wiring. Next slice should target another narrow `stroke_draw()` seam or stop
@@ -806,26 +810,33 @@ Done Checks:
Use this shape when adding a new task: Use this shape when adding a new task:
````markdown ````markdown
### AREA-000 - Imperative Task Name ### STR-004 - Collapse Remaining Stroke Dispatch Glue
Status: Ready Status: Ready
Score: +N scorecard area Score: +1 renderer boundary and OpenGL parity
Debt: `DEBT-0000` Debt: `DEBT-0036`
Scope: exact files or directories Scope: `src/canvas.cpp`, `src/legacy_canvas_stroke_execution_services.h`,
`tests/paint_renderer/stroke_execution_tests.cpp`
Goal: Goal:
One paragraph describing the behavior or ownership change. Remove the last inline `Canvas::stroke_draw()` dispatch construction that is
still just wiring. Keep the live adapter owning concrete GL objects, but move
the remaining helper-builder glue into retained stroke-execution helpers or
delete it if the callsite can consume a smaller already-tested helper.
Done Checks: Done Checks:
- Binary, grep-able, or testable condition. - `Canvas::stroke_draw()` has no ad hoc `LegacyCanvasStroke*Dispatch` literal
- Debt log update condition. blocks for the remaining live-pass wiring.
- Validation condition. - The helper-builder coverage tests prove the remaining dispatch helpers route
the expected callbacks.
- `docs/modernization/debt.md` records the reduced stroke-dispatch surface.
Validation: Validation:
```powershell ```powershell
command ctest --preset desktop-fast --build-config Debug -R "pp_paint_renderer_stroke_execution" --output-on-failure
& 'C:\Program Files\Microsoft Visual Studio\18\Community\MSBuild\Current\Bin\MSBuild.exe' out\build\windows-msvc-default\tests\pp_paint_renderer_stroke_execution_tests.vcxproj /p:Configuration=Debug /p:Platform=x64
``` ```
```` ````