Skip to content

Separate binding code into individual module-specific bindings#60

Merged
seungsoo47 merged 15 commits intoflutter-tizen:mainfrom
seungsoo47:dev.tizen_interop.separate_bindingcode.260311
Mar 25, 2026
Merged

Separate binding code into individual module-specific bindings#60
seungsoo47 merged 15 commits intoflutter-tizen:mainfrom
seungsoo47:dev.tizen_interop.separate_bindingcode.260311

Conversation

@seungsoo47
Copy link
Copy Markdown
Contributor

@seungsoo47 seungsoo47 commented Mar 11, 2026

This change is a refactoring effort to modularize the tizen_interop binding structure, improving code management and maintainability.

  • Refactored the monolithic binding code into separate and independent binding codes for each Tizen 6.0 API module.
  • Created separate entrypoint header files & ffigen configuration files for each Tizen API module
  • Generated separate binding code for each Tizen 6.0 module.
  • Fix helper script bugs.
  • Add missing headers and libraries in tizen 6.5 and 9.0.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request is a significant refactoring to modularize the FFI bindings, which greatly improves maintainability. The changes in the integration tests correctly reflect this new modular structure. I've found one minor issue in the test logic for error handling which should be addressed.

Comment thread example/integration_test/tizen_interop_test.dart
@JSUYA
Copy link
Copy Markdown
Member

JSUYA commented Mar 25, 2026

I tried to create 6.5 bindings based on this PR.
However, I believe a script should be provided that allows running scripts/ffigen_helper.sh 6.5 to generate ffigen-*.yaml in a basic format, similar to 6.0. Please let me know if I am missing anything.
I think the structure should allow other users to create new version bindings without issues and easy by following the guide.

@seungsoo47
Copy link
Copy Markdown
Contributor Author

I tried to create 6.5 bindings based on this PR. However, I believe a script should be provided that allows running scripts/ffigen_helper.sh 6.5 to generate ffigen-*.yaml in a basic format, similar to 6.0. Please let me know if I am missing anything. I think the structure should allow other users to create new version bindings without issues and easy by following the guide.

As we discussed offline, we will be able to provide the script later.

@seungsoo47 seungsoo47 merged commit 668638b into flutter-tizen:main Mar 25, 2026
2 checks passed
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