#development
- p
plain-leather-21357
09/29/2017, 4:47 PM@average-vr-56795 ohh that makes sense, I was wondering a bit about the ‘any’s. sounds good re: sync up next week! thanks - w
witty-crayon-22786
09/29/2017, 4:50 PM(can you fetch a link to what you mean by "Any" in this context? literally a protobuf type named
?)Anya- 2
- 2
- w
witty-crayon-22786
09/29/2017, 4:53 PMi was using threads heavily there because of scrollback, but now that we're caught up, let's forget about those, heh - a
average-vr-56795
09/29/2017, 4:53 PMOk 🙂 - a
average-vr-56795
09/29/2017, 4:53 PMThe reason I ended up here is I need to update futures for grpc, and there has since been a released version which has the PR mentioned in https://github.com/pantsbuild/pants/blob/master/src/rust/engine/Cargo.toml#L24 but it yields ALL the deprecation warnings 🙂 - a
average-vr-56795
09/29/2017, 4:54 PMhttps://github.com/alexcrichton/futures-rs/issues/228 is the context - w
witty-crayon-22786
09/29/2017, 4:56 PMaha, alrighty then - w
witty-crayon-22786
09/29/2017, 4:57 PMfire away! - w
witty-crayon-22786
09/29/2017, 4:57 PM@average-vr-56795: but... what you might do to minimize impact would be to re-add the method? - w
witty-crayon-22786
09/29/2017, 4:57 PMthe actual usage is much nicer than
, afaictBox::new - a
average-vr-56795
09/29/2017, 4:58 PMSure, I can give that a try 🙂 I guess I'll probably need to rename it because signature conflicts - w
witty-crayon-22786
09/29/2017, 4:59 PMyea - w
witty-crayon-22786
09/29/2017, 4:59 PMi think i actually experimented with doing that at some point... should have followed through to commit it. - a
average-vr-56795
09/29/2017, 5:00 PM(I've been learning many rust things today, it's been fun! I've done multi-threaded data sharing, some cute type things, some ownership things I hadn't come across yet...) - w
witty-crayon-22786
09/29/2017, 5:00 PMwoot. - w
witty-crayon-22786
09/29/2017, 5:00 PMany rough edges you want to talk about? - w
witty-crayon-22786
09/29/2017, 5:00 PM(other than the boxing, which annoys me as well) - a
average-vr-56795
09/29/2017, 5:01 PMYeah, boxing is a bit of a pain... And I continue to not like how I need to hoist variables out of cute one-liners because temporary scopes - a
average-vr-56795
09/29/2017, 5:01 PMBut generally, all pretty good 🙂 - w
witty-crayon-22786
09/29/2017, 5:01 PM@average-vr-56795: what's an example of the cute one-liner resistance? - w
witty-crayon-22786
09/29/2017, 5:02 PM@average-vr-56795: oh, and is there anything we should be landing right now? can talk while CI runs - a
average-vr-56795
09/29/2017, 5:02 PMOoh yes please! I meant to ask! - w
witty-crayon-22786
09/29/2017, 5:02 PM(looking at PRs now) - a
average-vr-56795
09/29/2017, 5:02 PMBoth of my open PRs are green and landable - a
average-vr-56795
09/29/2017, 5:03 PMOne includes the commit of the other - a
average-vr-56795
09/29/2017, 5:03 PMI don't know whether that means we need to land one, then rebase, then land the other, or whether github has a nice way of landing them both as separate commits without a merge commit - a
average-vr-56795
09/29/2017, 5:06 PMfoo.and_then(|arcmutex| arcmutex.deref().lock().unwrap().get(|hashmap| hashmap.get(bar)).and_then(|value| do_something_with(value)) I need to pull out hashmap as a variable, because otherwise I lose my lock at the end of the and_then scope and can't access value. - w
witty-crayon-22786
09/29/2017, 5:07 PMcomments on https://github.com/pantsbuild/pants/pull/4912 - a
average-vr-56795
09/29/2017, 5:07 PMFeels like a thing you could sugar away by having the and_then(|value|...) closure have a pointer to an anonymous temporary variable if you wanted, but they don't, so I need to pull out a named variablew- 2
- 11
- a
average-vr-56795
09/29/2017, 5:10 PMI guess today I get to learn how to start a weezy docker image locally to build a protoc 🙂