OGRECave/ogre

Prevent static initialization order fiasco for all class constants

Open

#3,321 opened on Apr 14, 2025

View on GitHub
 (0 comments) (0 reactions) (0 assignees)C++ (1,032 forks)auto 404
good first issuepapercut

Repository metrics

Stars
 (4,617 stars)
PR merge metrics
 (PR metrics pending)

Description

There is a possibility of static initialization order fiasco when the order in which objects with static storage duration are initialized is ambiguous (see example code to reproduce the problem). Several classes were updated recently to use constexpr for initializing static constants, and these fixes should be applied to other class constants as well:

Contributor guide

Research direction
Examine the listed headers (AxisAlignedBox.h, OgreMatrix3.h, etc.) and make the static constants constexpr. Follow the pattern from the linked PRs.
Tech stack
cpp
Domain
computer graphics
Issue type
Bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-2 days
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
C++constexpr
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70