Skip to content

Union support #61

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 4 commits into from
Sep 23, 2016
Merged

Union support #61

merged 4 commits into from
Sep 23, 2016

Conversation

emilio
Copy link
Contributor

@emilio emilio commented Sep 22, 2016

Built on top of #58, this adds support for untagged unions if not running under the --no-unstable-rust-flag.

The last commit is an example try run passing with this, but should be reverted (and the --no-unstable-rust flag passed by default to our harness).

This depends on serde-deprecated/aster#109.

r? @nox

@highfive
Copy link

warning Warning warning

  • These commits modify unsafe code. Please review it carefully!

@dtolnay
Copy link
Member

dtolnay commented Sep 23, 2016

I released aster 0.28.0 containing your union and array builders.

@emilio
Copy link
Contributor Author

emilio commented Sep 23, 2016

Thanks! This is ready for review then. r? @nox

@nox
Copy link
Contributor

nox commented Sep 23, 2016

Looks good! Just squash what should be squashed, otherwise r=me.

@emilio
Copy link
Contributor Author

emilio commented Sep 23, 2016

@bors-servo: r=nox

@bors-servo
Copy link

📌 Commit 9e1ddf9 has been approved by nox

@bors-servo
Copy link

⌛ Testing commit 9e1ddf9 with merge 978846c...

bors-servo pushed a commit that referenced this pull request Sep 23, 2016
Union support

Built on top of #58, this adds support for untagged unions if not running under the `--no-unstable-rust-flag`.

The last commit is an example try run passing with this, but should be reverted (and the `--no-unstable-rust` flag passed by default to our harness).

This depends on serde-deprecated/aster#109.

r? @nox
@bors-servo
Copy link

☀️ Test successful - status-travis

@bors-servo bors-servo merged commit 9e1ddf9 into rust-lang:master Sep 23, 2016
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants