Skip to content

Refactor Application singleton to use protected constructor and logic_error check - #23

Merged
Konstantysz merged 3 commits into
mainfrom
fix/issue-20-application-singleton
Feb 16, 2026
Merged

Refactor Application singleton to use protected constructor and logic_error check#23
Konstantysz merged 3 commits into
mainfrom
fix/issue-20-application-singleton

Conversation

@Konstantysz

Copy link
Copy Markdown
Owner

Fixes #20.

Changes:

  • Moved Application constructor to protected to prevent direct instantiation.
  • Added std::logic_error check to prevent multiple instances.
  • Verified with unit tests.

@github-actions

github-actions Bot commented Feb 16, 2026

Copy link
Copy Markdown

📊 Test Coverage Report

Coverage Statistics

Metric Coverage
Lines 58.88%
Functions 54.97%

@Konstantysz
Konstantysz merged commit 1726535 into main Feb 16, 2026
12 checks passed
@Konstantysz
Konstantysz deleted the fix/issue-20-application-singleton branch February 16, 2026 22:01
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.

Fix Architecture - Singleton / Static Global Mess in Application class

1 participant