Update proc-macro2, syn and quote to 1.0

[hobofan]

CLOSES #590

I also tried to lift the recursion limits in the examples, but it looks like they are still required (maybe due to macros from stdweb?).

[jstarry]

Great, thanks for jumping on this! That's a bummer that the recursion limit didn't go down :/

[jstarry]

We actually can't handle all Pat types. I think we only want to cover Pat::Ident and Pat::Wild. Here's a code snippet only handling Ident:

let var = match inputs.first().unwrap() {
  Pat::Ident(ident) => Ok(ident),
   _ => Err(syn::Error::new_spanned(or_span, "unsupported closure argument")),
}?;

[jstarry]

How about trying to avoid the to_owned allocation and use a helper fn like this:

fn match_path(path: &Path, id_str: &str) -> bool {
    if let Some(ident) = path.get_ident() {
        ident == id_str
    } else {
        false
    }
}

[hobofan]

I just found Path::is_ident(), which does that

[jstarry]

In order to fix the macro tests you will have to follow these steps:

1. Remove *.stderr files for the failing tets
2. Run cargo test --test macro_test
3. Copy the new generated *.stderr files from the wip dir into the test dir

[hobofan]

@jstarry Thanks for the feedback! Ready for review again.

[jstarry]

Looks great! Good find with is_ident :)

[jstarry]

bors r+

[bors]

Build succeeded

• continuous-integration/travis-ci/push