Skip to content

build(tools): fix format specifies type error on arm32#1089

Merged
ksqsf merged 1 commit intorime:masterfrom
WhiredPlanck:multiarch-tools
Sep 30, 2025
Merged

build(tools): fix format specifies type error on arm32#1089
ksqsf merged 1 commit intorime:masterfrom
WhiredPlanck:multiarch-tools

Conversation

@WhiredPlanck
Copy link
Contributor

Pull request

Issue tracker

Fixes will automatically close the related issue

Fixes # N/A

Feature

Describe feature of pull request

Unit test

  • Done

Manual test

  • Done

Code Review

  1. Unit and manual test pass
  2. GitHub Action CI pass
  3. At least one contributor reviews and votes
  4. Can be merged clean without conflicts
  5. PR will be merged by rebase upstream base

Additional Info

@eagleoflqj eagleoflqj requested a review from Copilot September 30, 2025 11:23
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes format specifier type errors on ARM32 by changing %lu to %zu in printf statements. The ARM32 architecture has different type sizes that cause compilation warnings/errors when using incorrect format specifiers for size_t values.

Key Changes

  • Updated two printf format specifiers from %lu to %zu for proper size_t formatting
  • Ensures cross-platform compatibility, particularly for ARM32 builds

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@ksqsf ksqsf merged commit 0a27e35 into rime:master Sep 30, 2025
10 checks passed
@WhiredPlanck WhiredPlanck deleted the multiarch-tools branch October 1, 2025 02:34
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.

3 participants