Hello Josh
We are trying to fix the errors we got in adapter.rs. We fixed most of
them.
The errors that are left are due to the changes we made to the init
functions.
We are not sure about the following errors, could you help us with this?
error[E0407]: method `get_modalias` is not a member of trait
`BluetoothAdapter`
--> src\adapter.rs:335:6
|
335 | / fn get_modalias(&self) -> Result<(String, u32, u32, u32),
Box<Error>> {
336 | | get_inner_and_call!(self, BluetoothAdapter, get_modalias)
337 | | }
| |_____^ not a member of trait `BluetoothAdapter`
error[E0277]: the size for values of type `(dyn adapter::BluetoothAdapter +
'static)` cannot be known at compilation time
--> src\adapter.rs:156:6
|
156 | / fn new() -> Result<BluetoothAdapter, Box<Error>> {
157 | | let adapter = try!(BluetoothAdapterEmpty::init());
158 | | Ok(BluetoothAdapter::Empty(Arc::new(adapter)))
159 | | }
| |_____^ doesn't have a size known at compile-time
-Niveditha Shankar
On Sat, Mar 30, 2019 at 1:09 PM Josh Bowman-Matthews <j...@joshmatthews.net>
wrote:
> Yes - use a Cargo override to build Servo with your local modified
> version of the devices crate. You can read more about overrides at
>
> https://doc.rust-lang.org/cargo/reference/specifying-dependencies.html#overriding-dependencies
> and
> https://doc.rust-lang.org/cargo/reference/manifest.html#the-patch-section
> and you can see where to add this at
> https://github.com/servo/servo/blob/master/Cargo.toml#L17-L26 . Once
> Servo builds with the modified crate, you can run the automated
> bluetooth tests to verify that the behaviour is the same: `./mach
> test-wpt tests/wpt/mozilla/tests/bluetooth`.
>
> Cheers,
> Josh
>
> On 3/30/19 12:50 PM, Niveditha Shankar wrote:
> > Okay I think I get it. Thank you!
> >
> > Is there anyway I can check if the changes I make to the devices crate is
> > right?
> >
> > Our submission is on Monday and we are going to continue working with
> > Webbluetooth for our final project too. So if our implementation of our
> > adapter.rs file is right, we can go ahead and do something similar for
> the
> > rest of the enum values. It would be really helpful if there is a way to
> > check that.
> >
> >
> > Niveditha Shankar
> _______________________________________________
> dev-servo mailing list
> dev-servo@lists.mozilla.org
> https://lists.mozilla.org/listinfo/dev-servo
>
_______________________________________________
dev-servo mailing list
dev-servo@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-servo
