Skip to content

Conversation

erickt
Copy link
Contributor

@erickt erickt commented Jan 24, 2013

Good morning,

This patch converts most of the libcore and libstd records into structs. I believe all that's left is extfmt and pipes, which needs a compiler change. It also removes record support for auto-encode, with the idea of simplifying down it down for eventual merger with the deriving macro. Finally, it includes what I hope is a fix for the randomred time tests by merging all those tests into one call so there are no more races on tzset.

@catamorphism
Copy link
Contributor

Great! I want to run this on try just to be sure, but I'd love to see an end to the ignored time tests.

@erickt
Copy link
Contributor Author

erickt commented Jan 25, 2013

I've updated this to remove all the records I could find. Once we make a snapshot, I believe we'll be able to start removing record support from rustc. Goodbye fair {x: int, y: int}, I knew ye well...

@catamorphism
Copy link
Contributor

Yes, I'm working on disabling records by default and adding a legacy_records attribute at this moment. In the meantime, I'm just waiting for the FreeBSD build to finish running on try, then I'll merge this.

catamorphism added a commit that referenced this pull request Jan 25, 2013
convert most of libcore and libstd to structs, work around tzset race
@catamorphism catamorphism merged commit 9898485 into rust-lang:incoming Jan 25, 2013
@catamorphism
Copy link
Contributor

And it's green! Thanks!

RalfJung added a commit to RalfJung/rust that referenced this pull request Oct 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants