Skip to content

Conversation

@jorenham
Copy link
Owner

@jorenham jorenham commented Sep 8, 2025

this should make it easier to more custom c++ wrapper functions

Copilot AI review requested due to automatic review settings September 8, 2025 03:21
@jorenham jorenham added this to the 0.1.0 milestone Sep 8, 2025
Copy link
Contributor

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 restructures the build.rs file to improve organization and maintainability of C++ wrapper function generation. The changes move inline string literals to constants and refactor helper functions for better modularity.

Key changes:

  • Extract inline C++ code strings to named constants (DECL_, IMPL_)
  • Refactor formatting functions to use a more functional programming style
  • Consolidate allowlist generation into a dedicated function

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

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@jorenham jorenham merged commit 96d9e23 into master Sep 8, 2025
7 checks passed
@jorenham jorenham deleted the build.rs-cleanup branch September 8, 2025 03:32
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.

1 participant