Skip to content

Fix unused parameter warnings in HostPlatformViewTraitsInitializer.h (#56491)#56491

Closed
christophpurrer wants to merge 1 commit intofacebook:mainfrom
christophpurrer:export-D101108434
Closed

Fix unused parameter warnings in HostPlatformViewTraitsInitializer.h (#56491)#56491
christophpurrer wants to merge 1 commit intofacebook:mainfrom
christophpurrer:export-D101108434

Conversation

@christophpurrer
Copy link
Copy Markdown
Contributor

@christophpurrer christophpurrer commented Apr 18, 2026

Summary:

Fixed clang-diagnostic-unused-parameter warnings in HostPlatformViewTraitsInitializer.h by commenting out unused parameter names while maintaining API compatibility.

Changes:

  • formsStackingContext: Commented out unused 'props' parameter
  • formsView: Commented out unused 'props' parameter
  • isKeyboardFocusable: Already had parameter commented out correctly

Changelog: [Internal]

Reviewed By: zeyap

Differential Revision: D101108434

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Apr 18, 2026
@meta-codesync
Copy link
Copy Markdown

meta-codesync Bot commented Apr 18, 2026

@christophpurrer has exported this pull request. If you are a Meta employee, you can view the originating Diff in D101108434.

@meta-codesync meta-codesync Bot changed the title Fix unused parameter warnings in HostPlatformViewTraitsInitializer.h Fix unused parameter warnings in HostPlatformViewTraitsInitializer.h (#56491) Apr 20, 2026
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Apr 20, 2026
…acebook#56491)

Summary:

Fixed clang-diagnostic-unused-parameter warnings in HostPlatformViewTraitsInitializer.h by commenting out unused parameter names while maintaining API compatibility.

Changes:
- `formsStackingContext`: Commented out unused 'props' parameter
- `formsView`: Commented out unused 'props' parameter
- `isKeyboardFocusable`: Already had parameter commented out correctly

Differential Revision: D101108434
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Apr 20, 2026
…acebook#56491)

Summary:

Fixed clang-diagnostic-unused-parameter warnings in HostPlatformViewTraitsInitializer.h by commenting out unused parameter names while maintaining API compatibility.

Changes:
- `formsStackingContext`: Commented out unused 'props' parameter
- `formsView`: Commented out unused 'props' parameter
- `isKeyboardFocusable`: Already had parameter commented out correctly

Differential Revision: D101108434
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Apr 20, 2026
…acebook#56491)

Summary:
Pull Request resolved: facebook#56491

Fixed clang-diagnostic-unused-parameter warnings in HostPlatformViewTraitsInitializer.h by commenting out unused parameter names while maintaining API compatibility.

Changes:
- `formsStackingContext`: Commented out unused 'props' parameter
- `formsView`: Commented out unused 'props' parameter
- `isKeyboardFocusable`: Already had parameter commented out correctly

Differential Revision: D101108434
@christophpurrer christophpurrer force-pushed the export-D101108434 branch 2 times, most recently from 5596077 to a9add81 Compare April 20, 2026 01:26
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Apr 20, 2026
…acebook#56491)

Summary:
Pull Request resolved: facebook#56491

Fixed clang-diagnostic-unused-parameter warnings in HostPlatformViewTraitsInitializer.h by commenting out unused parameter names while maintaining API compatibility.

Changes:
- `formsStackingContext`: Commented out unused 'props' parameter
- `formsView`: Commented out unused 'props' parameter
- `isKeyboardFocusable`: Already had parameter commented out correctly

Differential Revision: D101108434
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Apr 20, 2026
…acebook#56491)

Summary:

Fixed clang-diagnostic-unused-parameter warnings in HostPlatformViewTraitsInitializer.h by commenting out unused parameter names while maintaining API compatibility.

Changes:
- `formsStackingContext`: Commented out unused 'props' parameter
- `formsView`: Commented out unused 'props' parameter
- `isKeyboardFocusable`: Already had parameter commented out correctly

Reviewed By: zeyap

Differential Revision: D101108434
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Apr 20, 2026
…acebook#56491)

Summary:

Fixed clang-diagnostic-unused-parameter warnings in HostPlatformViewTraitsInitializer.h by commenting out unused parameter names while maintaining API compatibility.

Changes:
- `formsStackingContext`: Commented out unused 'props' parameter
- `formsView`: Commented out unused 'props' parameter
- `isKeyboardFocusable`: Already had parameter commented out correctly

Reviewed By: zeyap

Differential Revision: D101108434
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Apr 20, 2026
…acebook#56491)

Summary:
Pull Request resolved: facebook#56491

Fixed clang-diagnostic-unused-parameter warnings in HostPlatformViewTraitsInitializer.h by commenting out unused parameter names while maintaining API compatibility.

Changes:
- `formsStackingContext`: Commented out unused 'props' parameter
- `formsView`: Commented out unused 'props' parameter
- `isKeyboardFocusable`: Already had parameter commented out correctly

Reviewed By: zeyap

Differential Revision: D101108434
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Apr 20, 2026
…acebook#56491)

Summary:
Pull Request resolved: facebook#56491

Fixed clang-diagnostic-unused-parameter warnings in HostPlatformViewTraitsInitializer.h by commenting out unused parameter names while maintaining API compatibility.

Changes:
- `formsStackingContext`: Commented out unused 'props' parameter
- `formsView`: Commented out unused 'props' parameter
- `isKeyboardFocusable`: Already had parameter commented out correctly

Reviewed By: zeyap

Differential Revision: D101108434
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Apr 20, 2026
…acebook#56491)

Summary:

Fixed clang-diagnostic-unused-parameter warnings in HostPlatformViewTraitsInitializer.h by commenting out unused parameter names while maintaining API compatibility.

Changes:
- `formsStackingContext`: Commented out unused 'props' parameter
- `formsView`: Commented out unused 'props' parameter
- `isKeyboardFocusable`: Already had parameter commented out correctly


Changelog: [Internal]

Reviewed By: zeyap

Differential Revision: D101108434
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Apr 20, 2026
…acebook#56491)

Summary:

Fixed clang-diagnostic-unused-parameter warnings in HostPlatformViewTraitsInitializer.h by commenting out unused parameter names while maintaining API compatibility.

Changes:
- `formsStackingContext`: Commented out unused 'props' parameter
- `formsView`: Commented out unused 'props' parameter
- `isKeyboardFocusable`: Already had parameter commented out correctly


Changelog: [Internal]

Reviewed By: zeyap

Differential Revision: D101108434
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Apr 20, 2026
…acebook#56491)

Summary:

Fixed clang-diagnostic-unused-parameter warnings in HostPlatformViewTraitsInitializer.h by commenting out unused parameter names while maintaining API compatibility.

Changes:
- `formsStackingContext`: Commented out unused 'props' parameter
- `formsView`: Commented out unused 'props' parameter
- `isKeyboardFocusable`: Already had parameter commented out correctly


Changelog: [Internal]

Reviewed By: zeyap

Differential Revision: D101108434
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Apr 20, 2026
…acebook#56491)

Summary:
Pull Request resolved: facebook#56491

Fixed clang-diagnostic-unused-parameter warnings in HostPlatformViewTraitsInitializer.h by commenting out unused parameter names while maintaining API compatibility.

Changes:
- `formsStackingContext`: Commented out unused 'props' parameter
- `formsView`: Commented out unused 'props' parameter
- `isKeyboardFocusable`: Already had parameter commented out correctly

Changelog: [Internal]

Reviewed By: zeyap

Differential Revision: D101108434
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Apr 20, 2026
…acebook#56491)

Summary:
Pull Request resolved: facebook#56491

Fixed clang-diagnostic-unused-parameter warnings in HostPlatformViewTraitsInitializer.h by commenting out unused parameter names while maintaining API compatibility.

Changes:
- `formsStackingContext`: Commented out unused 'props' parameter
- `formsView`: Commented out unused 'props' parameter
- `isKeyboardFocusable`: Already had parameter commented out correctly

Changelog: [Internal]

Reviewed By: zeyap

Differential Revision: D101108434
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Apr 21, 2026
…acebook#56491)

Summary:

Fixed clang-diagnostic-unused-parameter warnings in HostPlatformViewTraitsInitializer.h by commenting out unused parameter names while maintaining API compatibility.

Changes:
- `formsStackingContext`: Commented out unused 'props' parameter
- `formsView`: Commented out unused 'props' parameter
- `isKeyboardFocusable`: Already had parameter commented out correctly


Changelog: [Internal]

Reviewed By: zeyap

Differential Revision: D101108434
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Apr 21, 2026
…acebook#56491)

Summary:
Pull Request resolved: facebook#56491

Fixed clang-diagnostic-unused-parameter warnings in HostPlatformViewTraitsInitializer.h by commenting out unused parameter names while maintaining API compatibility.

Changes:
- `formsStackingContext`: Commented out unused 'props' parameter
- `formsView`: Commented out unused 'props' parameter
- `isKeyboardFocusable`: Already had parameter commented out correctly

Changelog: [Internal]

Reviewed By: zeyap

Differential Revision: D101108434
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Apr 21, 2026
…acebook#56491)

Summary:

Fixed clang-diagnostic-unused-parameter warnings in HostPlatformViewTraitsInitializer.h by commenting out unused parameter names while maintaining API compatibility.

Changes:
- `formsStackingContext`: Commented out unused 'props' parameter
- `formsView`: Commented out unused 'props' parameter
- `isKeyboardFocusable`: Already had parameter commented out correctly


Changelog: [Internal]

Reviewed By: zeyap

Differential Revision: D101108434
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Apr 21, 2026
…acebook#56491)

Summary:
Pull Request resolved: facebook#56491

Fixed clang-diagnostic-unused-parameter warnings in HostPlatformViewTraitsInitializer.h by commenting out unused parameter names while maintaining API compatibility.

Changes:
- `formsStackingContext`: Commented out unused 'props' parameter
- `formsView`: Commented out unused 'props' parameter
- `isKeyboardFocusable`: Already had parameter commented out correctly

Changelog: [Internal]

Reviewed By: zeyap

Differential Revision: D101108434
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Apr 21, 2026
…acebook#56491)

Summary:

Fixed clang-diagnostic-unused-parameter warnings in HostPlatformViewTraitsInitializer.h by commenting out unused parameter names while maintaining API compatibility.

Changes:
- `formsStackingContext`: Commented out unused 'props' parameter
- `formsView`: Commented out unused 'props' parameter
- `isKeyboardFocusable`: Already had parameter commented out correctly


Changelog: [Internal]

Reviewed By: zeyap

Differential Revision: D101108434
…acebook#56491)

Summary:
Pull Request resolved: facebook#56491

Fixed clang-diagnostic-unused-parameter warnings in HostPlatformViewTraitsInitializer.h by commenting out unused parameter names while maintaining API compatibility.

Changes:
- `formsStackingContext`: Commented out unused 'props' parameter
- `formsView`: Commented out unused 'props' parameter
- `isKeyboardFocusable`: Already had parameter commented out correctly

Changelog: [Internal]

Reviewed By: zeyap

Differential Revision: D101108434
@react-native-bot
Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @christophpurrer in eed1e34

When will my fix make it into a release? | How to file a pick request?

@react-native-bot react-native-bot added the Merged This PR has been merged. label Apr 22, 2026
@meta-codesync
Copy link
Copy Markdown

meta-codesync Bot commented Apr 22, 2026

This pull request has been merged in eed1e34.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants