Skip to content

Conversation

@yoshinorisano
Copy link

There are a lot of compiler warnings when building hound:

warning: use of deprecated macro `try`: use the `?` operator instead

So I tried to fix that. Does this make sense to you?

@yoshinorisano
Copy link
Author

@ruuda
Hi! Could you review this?

@atoktoto
Copy link

It would be great if this got merged. I'm working on a no_std support for this library. While I don't expect my changes will ever get merged (I had to switch to nightly) having this accepted would make it easier for me to keep my fork in sync with future changes.

@maxtimbo
Copy link

@ruuda Can this be merged?

@bbb651
Copy link

bbb651 commented Sep 24, 2024

Haven't checked the open PRs and I redid this myself in #83, along with some other things, would be nice to get them merged. I noticed there's this repo and an identical codeberg link to each other, I'm not sure who's the mirror of who, but they both seem inactive

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.

4 participants