Skip to content

Follow core's rename of format to unsafe-format - #21

Draft
hellerve wants to merge 1 commit into
masterfrom
unsafe-format
Draft

Follow core's rename of format to unsafe-format#21
hellerve wants to merge 1 commit into
masterfrom
unsafe-format

Conversation

@hellerve

@hellerve hellerve commented Sep 5, 2026

Copy link
Copy Markdown
Member

Carp core is renaming the format interface to unsafe-format (carp-lang/Carp#1432: format is a thin single-argument printf wrapper with no check that the string carries exactly one directive). Merge after that lands; until then CI is red because unsafe-format does not exist yet.

  • CLI.Type.format becomes CLI.Type.unsafe-format and implements unsafe-format. It takes a caller-supplied &String and forwards it to printf unchecked, so it inherits the hazard in full; a comment now says so
  • Long.format / Double.format / String.format call sites follow
  • reflowed by carp-fmt where the longer name crossed the wrap column

204/204 green against a local core carrying the rename.

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.

1 participant