8

Add the `--override-abi` option by pvdrz · Pull Request #2329 · rust-lang/rust-b...

 1 year ago
source link: https://github.com/rust-lang/rust-bindgen/pull/2329
Go to the source link to view the article. You can view the picture content, updated content and better typesetting reading experience. If the link is broken, please click the button below to view the snapshot at that time.
neoserver,ios ssh client

Conversation

Collaborator

@pvdrz pvdrz commented Nov 1, 2022

This option can be used from the CLI with the : syntax and it overrides the ABI of a function if it matches .

Fixes #2257 as the ".*" regex can be used to match every function.

All reactions

Collaborator

@emilio emilio left a comment

r=me with that simplification.

All reactions

Collaborator

Author

pvdrz commented Nov 2, 2022 •

edited

@emilio before we move forward with this, we have this question in #2328 where eventually people would like to have another option with the shape <extra_arg><separator><regex>. Maybe we should agree on the separator before moving forward? I used : here but for the other use of this : won't work because <extra_arg> contains :.

Another suggested option is / but I don't know if that's clear enough

pvdrz

merged commit 9c32b46 into

rust-lang:master

Nov 2, 2022

24 checks passed

pvdrz

deleted the override-abi branch

Nov 2, 2022

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

Reviewers

emilio

Assignees

No one assigned

Labels
None yet
Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

default ABI

2 participants

report

Recommend

About Joyk


Aggregate valuable and interesting links.
Joyk means Joy of geeK