Skip to content

Conversation

@PeterN
Copy link
Owner

@PeterN PeterN commented May 14, 2024

Motivation / Problem

Description

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 save game format? (label 'savegame upgrade')
  • 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 force-pushed the fontsearcher branch 3 times, most recently from db0431b to 85cc11f Compare May 14, 2024 21:59

/* If argv is a number treat it as an index into the list of fonts. */
int index;
auto [_, err] = std::from_chars(family.data(), family.data() + family.size(), index, 10);

Check notice

Code scanning / CodeQL

Unused local variable

Variable _ is not used.
@PeterN PeterN force-pushed the fontsearcher branch 3 times, most recently from f303103 to edafea5 Compare May 16, 2024 10:11
@PeterN PeterN force-pushed the fontsearcher branch 2 times, most recently from 5d379b6 to 60d9bf2 Compare June 15, 2024 20:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants