klingtnet / rosc

An OSC library for Rust.
Apache License 2.0
173 stars 25 forks source link

Possible small bug? #39

Closed aleixjf closed 1 year ago

aleixjf commented 1 year ago

Hello,

I've found an error while compiling after having upgraded to rosc 0.9.1 from version 0.8.

Compiling rosc v0.9.1
error[E0449]: unnecessary visibility qualifier
   --> /Users/<my-username>/.cargo/registry/src/github.com-1ecc6299db9ec823/rosc-0.9.1/src/types.rs:106:5
    |
106 |     pub fn from(time: (u32, u32)) -> OscTime {
    |     ^^^ `pub` not permitted here because it's implied

I've temporarily solved it by following the suggested solution and removing the pub modifier.

klingtnet commented 1 year ago

Hi!

Thanks for reporting the issue. I wasn't able to reproduce this problem, which Rust version did you use?

klingtnet commented 1 year ago

There's also no pub modifier assigned to this function in rosc 0.9.1:

$ git describe --tags
v0.9.1
$ rg -A3 --fixed-strings 'fn from(time: (u32, u32)) -> Self' src/types.rs
198:    fn from(time: (u32, u32)) -> Self {
199-        OscType::Time(time.into())
200-    }
201-}

🤔

klingtnet commented 1 year ago

I think this can be closed since there's no pub modifier here. The issue might have been a false positive. Please reopen if this error occurs again.