Skip to content

Conversation

@jrasky
Copy link
Contributor

@jrasky jrasky commented Jan 28, 2015

Fix for breaking lifetime changes introduced on the 27th.

Of note: tests do pass, but only build if linked against a libregex and a libregex_macros that wasn't built as a test. Building the tests against a libregex_macros built with --test causes Rust to complain about regex($re) to not live long enough.

I'm not entirely sure as to the cause, but given compiler plugins are experimental it isn't entirely surprising.

Creating nlist and clist with &mut Threads::new causes Rust to complain. Create them without pointer indirection at first, and change corresponding function calls to reference them as pointers instead.
@alexcrichton
Copy link
Member

I'm a little confused by github right now, it says this can merge cleanly but I think I already fixed this in 6c0f8aa, so I'm not totally sure what's going on!

Regardless, I think this fix is already in master, so just be sure to run cargo update to get the latest version, thanks anyway though!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants