Skip to content

Controlliing inlining - #79

Merged
dewert99 merged 3 commits into
hobofan:masterfrom
vigna:inline
Apr 19, 2026
Merged

dewert99 merged 3 commits into
hobofan:masterfrom
vigna:inline

Conversation

@vigna

@vigna vigna commented Apr 18, 2026

Copy link
Copy Markdown
Contributor

This is a proposal for controlling the inlining policy. There's a global setting (default: as now) and a local override for maximum flexibility.

I'm having some problems to compile the tests—I guess it's a compiler-version problem, but ATM trying to use the +1.68 toolchain gives me HTTP errors.

@vigna

vigna commented Apr 18, 2026

Copy link
Copy Markdown
Contributor Author

There's some docs in the README.

@vigna

vigna commented Apr 18, 2026

Copy link
Copy Markdown
Contributor Author

All actions green. This should be it.

@vigna

vigna commented Apr 19, 2026

Copy link
Copy Markdown
Contributor Author

You might want to move the position of the section about inlining in the README (and I'm not a native speaker, so you might want to change the wording). But the content I think is ok. I'm using this now in sux and it's working like a charm.

@dewert99
dewert99 merged commit 0990547 into hobofan:master Apr 19, 2026
4 checks passed
@dewert99

Copy link
Copy Markdown
Collaborator

Thanks

@vigna

vigna commented Apr 19, 2026

Copy link
Copy Markdown
Contributor Author

Any chance of a release soon? 😬

@dewert99

Copy link
Copy Markdown
Collaborator

I just released version 0.5.1 which contains your changes with a tweak I made so that it can more easily support become in the future.

@vigna

vigna commented Apr 20, 2026

Copy link
Copy Markdown
Contributor Author

Thanks!

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