Skip to content

Restore scheduler declaration in calc_tables#225

Closed
tameware with Copilot wants to merge 1 commit into
developfrom
copilot/fix-review-comment-216
Closed

Restore scheduler declaration in calc_tables#225
tameware with Copilot wants to merge 1 commit into
developfrom
copilot/fix-review-comment-216

Conversation

Copilot AI commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

calc_tables.cpp no longer declared the global scheduler symbol after the parallelism refactor. That breaks DDS_SCHEDULER builds, where the timing macros and scheduler.PrintTiming() still require the translation unit to reference the global instance.

  • Build fix

    • Restore the missing forward declaration for the global scheduler in library/src/calc_tables.cpp.
  • Scope

    • No behavioral changes.
    • No changes outside the review comment being addressed.
extern Memory memory;
extern Scheduler scheduler;

Copilot AI changed the title [WIP] Fix code according to review comment 216 Restore scheduler declaration in calc_tables Jul 3, 2026
Copilot AI requested a review from tameware July 3, 2026 14:42
@tameware tameware closed this Jul 3, 2026
@tameware tameware deleted the copilot/fix-review-comment-216 branch July 3, 2026 14:44
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