andreastt

diff --git a/src/message.rs b/src/message.rs
index b11b66d..72f0215 100644
--- a/src/message.rs
+++ b/src/message.rs
@@ -1,10 +1,10 @@
-use serde::de::{self, SeqAccess, Visitor};
+use serde::de::{self, SeqAccess, Unexpected, Visitor};
 use serde::{Deserialize, Deserializer, Serialize, Serializer};
 use serde_json::{Map, Value};
 use serde_repr::{Deserialize_repr, Serialize_repr};

andreastt

[src/message.rs:313] serde_json::to_string(&json) = Ok(
    "[1,42,{\"error\":\"no such element\",\"message\":\"\",\"stacktrace\":\"\"},null]",
)
[src/message.rs:315] serde_json::to_string(&msg) = Ok(
    "[1,42,{\"error\":\"no such element\",\"message\":\"\",\"stacktrace\":\"\"},null]",
)

andreastt

% cargo tree -p serde_repr
serde_repr v0.1.4
├── proc-macro2 v0.4.30
│   └── unicode-xid v0.1.0
├── quote v0.6.12
│   └── proc-macro2 v0.4.30 (*)
└── syn v0.15.38
    ├── proc-macro2 v0.4.30 (*)
    ├── quote v0.6.12 (*)
    └── unicode-xid v0.1.0 (*)

andreastt

#![allow(dead_code)]
#![allow(unused_imports)]

use serde::de::{self, SeqAccess, Visitor};
use serde::{Deserialize, Deserializer, Serialize, Serializer};
use serde_json::{json, Map, Value};
use std::borrow::Cow;
use std::fmt;
use std::marker::PhantomData;

andreastt

 0:29.49     Blocking waiting for file lock on package cache lock
 0:32.24    Compiling lazy_static v1.2.0
 0:32.27 error: the option `Z` is only accepted on the nightly compiler
 0:32.32 error: Could not compile `lazy_static`.
 0:32.32 To learn more, run the command again with --verbose.
 0:32.32 make[4]: *** [/home/ato/src/gecko/config/makefiles/rust.mk:319: force-cargo-host-program-build] Error 101
 0:32.33 make[3]: *** [/home/ato/src/gecko/config/recurse.mk:74: js/src/frontend/binast/host] Error 2
 0:32.33 make[3]: *** Waiting for unfinished jobs....
 0:33.27    Compiling void v1.0.2
 0:33.31 error: the option `Z` is only accepted on the nightly compiler

andreastt

#![allow(dead_code)]
#![allow(unused_imports)]

use serde::de::{self, SeqAccess, Visitor};
use serde::{Deserialize, Deserializer, Serialize, Serializer};
use serde_json::{json, Map, Value};
use std::borrow::Cow;
use std::fmt;
use std::marker::PhantomData;

andreastt

diff --git a/src/message.rs b/src/message.rs
index aa98505..252a06d 100644
--- a/src/message.rs
+++ b/src/message.rs
@@ -125,7 +125,9 @@ impl<'de> Visitor<'de> for MessageVisitor {
                 let params: Params = match name {
                     Command::WebDriver(ref cmd) => match cmd {
                         FindElement => {
-                            Params::Locator(seq.next_element::<webdriver::Locator>()?.unwrap())
+                            let locator = seq.next_element::<webdriver::Locator>()?

andreastt

#![allow(dead_code)]
#![allow(unused_imports)]

use serde::de::{self, SeqAccess, Visitor};
use serde::{Deserialize, Deserializer, Serialize, Serializer};
use serde_repr::{Serialize_repr, Deserialize_repr};
use serde_json::{json, Map, Value};
use std::borrow::Cow;
use std::fmt;
use std::marker::PhantomData;

andreastt

 4:04.41 INFO leakcheck | Processing leak log file /tmp/tmpVnWT6P.mozrunner/runtests_leaks_tab_pid3866.log
 4:04.41 INFO 
 4:04.41 INFO == BloatView: ALL (cumulative) LEAK AND BLOAT STATISTICS, tab process 3866
 4:04.41 INFO 
 4:04.41 INFO      |<----------------Class--------------->|<-----Bytes------>|<----Objects---->|
 4:04.41 INFO      |                                      | Per-Inst   Leaked|   Total      Rem|
 4:04.41 INFO    0 |TOTAL                                 |       42    51436|   39249     2093|
 4:04.41 INFO    1 |APZEventState                         |      120      120|       1        1|
 4:04.41 INFO    7 |ActiveElementManager                  |       40       40|       1        1|
 4:04.41 INFO   21 |AtomSet                               |       32      448|      44       14|

andreastt

% ./mach lint -funix remote
New python executable in /home/ato/src/gecko/obj-x86_64-pc-linux-gnu/_virtualenvs/init/bin/python2.7
Not overwriting existing python script /home/ato/src/gecko/obj-x86_64-pc-linux-gnu/_virtualenvs/init/bin/python (you must use /home/ato/src/gecko/obj-x86_64-pc-linux-gnu/_virtualenvs/init/bin/python2.7)
Installing setuptools, pip, wheel...done.
running build_ext
copying build/lib.linux-x86_64-2.7/psutil/_psutil_linux.so -> psutil
copying build/lib.linux-x86_64-2.7/psutil/_psutil_posix.so -> psutil

Error processing command. Ignoring because optional. (optional:packages.txt:comm/build/virtualenv_packages.txt)