Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove platform macros from tip_text_service.cc #1132

Merged

Conversation

yukawa
Copy link
Collaborator

@yukawa yukawa commented Dec 3, 2024

Description

This commit removes the dependency on platform macros such as _M_X64 and _M_IX86 from tip_text_service.cc.

There must be no change in the final artifacts.

This is a preparation to build Mozc for ARM64 (#1130).

Issue IDs

Steps to test new behaviors (if any)

  • OS: Windows 11 23H2
  • Steps:
    1. Build and install Mozc64.msi
    2. Confirm you can type with Mozc on both 32-bit app and 64-bit app.

This commit removes the dependency on platform macros such as _M_X64 and
_M_IX86 from tip_text_service.cc.

There must be no change in the final artifacts.

This is a preparation to build Mozc for ARM64 (google#1130).
@yukawa yukawa force-pushed the support_arm64_in_tip_text_service branch from 2dd8273 to 1d3bf9b Compare December 4, 2024 11:13
@hiroyuki-komatsu hiroyuki-komatsu merged commit 0960a75 into google:master Dec 7, 2024
1 check passed
@hiroyuki-komatsu
Copy link
Collaborator

We have merged your PR.
Thank you for your contribution!

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