Skip to content

make internal FTSInfo take CInt#3586

Merged
Lukasa merged 1 commit intoapple:mainfrom
weissi:jw-intuint
May 5, 2026
Merged

make internal FTSInfo take CInt#3586
Lukasa merged 1 commit intoapple:mainfrom
weissi:jw-intuint

Conversation

@weissi
Copy link
Copy Markdown
Member

@weissi weissi commented May 5, 2026

Motivation:

Internally, FTSInfo always casted up to CInt from UInt16 but for some reason it only accepted UInt16. Let's add a CInt constructor too.

Modifications:

  • Make internal FTSInfo constructible from CInt and UInt16.

Result:

  • Slightly better platform compatibility
  • Possibly fewer casts

@Lukasa Lukasa added the 🔨 semver/patch No public API change. label May 5, 2026
@Lukasa Lukasa enabled auto-merge (squash) May 5, 2026 14:59
@Lukasa Lukasa merged commit 1c8b29c into apple:main May 5, 2026
51 of 55 checks passed
@weissi weissi deleted the jw-intuint branch May 5, 2026 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Lukasa Lukasa Lukasa approved these changes

Assignees

No one assigned

Labels

🔨 semver/patch No public API change.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@weissi @Lukasa