vincentdephily / emlop

EMerge LOg Parser
GNU General Public License v3.0
38 stars 4 forks source link

Cargo test fails, because too few arguments in new_hist(...) function call #12

Closed anaskaejdar closed 4 years ago

anaskaejdar commented 4 years ago

Here's the relevant part of the command output:

   Compiling serde_json v1.0.39
   Compiling failure v0.1.5
   Compiling assert_cli v0.6.3
   Compiling emlop v0.3.1 (/home/tasso/Documents/prog/git/emlop)
error[E0061]: this function takes 9 parameters but 8 parameters were supplied
   --> src/parser.rs:314:20
    |
51  | / pub fn new_hist<R: Read>(reader: R,
52  | |                          filename: String,
53  | |                          min_ts: Option<i64>,
54  | |                          max_ts: Option<i64>,
...   |
110 | |     Ok(rx)
111 | | }
    | |_- defined here
...
314 |           let hist = new_hist(File::open(filename).unwrap(),
    |  ____________________^
315 | |                             filename.into(),
316 | |                             filter_mints,
317 | |                             filter_maxts,
...   |
320 | |                             filter_pkg,
321 | |                             exact).unwrap();
    | |__________________________________^ expected 9 parameters

error: aborting due to previous error

For more information about this error, try `rustc --explain E0061`.
error: could not compile `emlop`.

To learn more, run the command again with --verbose.
➜  emlop git:(master)
vincentdephily commented 4 years ago

Thanks for the report ! Had forgotten to push some WIP. There's still a good bit of work to do around the unmerge feature.