Skip to content

Codechange: remove duplicate and out-of-date documentation for LoadFont#15411

Merged
PeterN merged 1 commit intoOpenTTD:masterfrom
PeterN:load-font-docs
Apr 2, 2026
Merged

Codechange: remove duplicate and out-of-date documentation for LoadFont#15411
PeterN merged 1 commit intoOpenTTD:masterfrom
PeterN:load-font-docs

Conversation

@PeterN
Copy link
Copy Markdown
Member

@PeterN PeterN commented Apr 2, 2026

Motivation / Problem

Duplicate and somewhat outdated documentation for FontCache() implementations, since FontCacheFactory was introduced.

Description

Remove duplicate and out-of-date documentation for LoadFont.
Move comments inline to code where useful.

Limitations

Checklist for review

Some things are not automated, and forgotten often. This list is a reminder for the reviewers.

  • The bug fix is important enough to be backported? (label: 'backport requested')
  • This PR touches english.txt or translations? Check the guidelines
  • This PR affects the GS/AI API? (label 'needs review: Script API')
    • ai_changelog.hpp, game_changelog.hpp need updating.
    • The compatibility wrappers (compat_*.nut) need updating.
  • This PR affects the NewGRF API? (label 'needs review: NewGRF')

@PeterN PeterN merged commit 36878bf into OpenTTD:master Apr 2, 2026
16 checks passed
@PeterN PeterN deleted the load-font-docs branch April 2, 2026 11:20
Baricus pushed a commit to Baricus/OpenTTD that referenced this pull request Apr 5, 2026
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