Skip to content
This repository has been archived by the owner on Jun 8, 2020. It is now read-only.

Derive Debug for (almost) all public types #212

Merged
merged 1 commit into from
Dec 14, 2015
Merged

Conversation

antrik
Copy link
Contributor

@antrik antrik commented Dec 13, 2015

Leaving out DrawTarget/DrawTargetBacking, as the latter includes
GLRasterizationContext from Skia, which is missing Debug.

(And I don't feel like doing this for Skia right now -- this is already
quite getting out of hand...)

Review on Reviewable

Leaving out DrawTarget/DrawTargetBacking, as the latter includes
GLRasterizationContext from Skia, which is missing Debug.

(And I don't feel like doing this for Skia right now -- this is already
quite getting out of hand...)
@KiChjang
Copy link

@bors-servo try

@bors-servo
Copy link

⌛ Trying commit 0a0c2e2 with merge 19ed46b...

bors-servo pushed a commit that referenced this pull request Dec 13, 2015
Derive Debug for (almost) all public types

Leaving out DrawTarget/DrawTargetBacking, as the latter includes
GLRasterizationContext from Skia, which is missing Debug.

(And I don't feel like doing this for Skia right now -- this is already
quite getting out of hand...)

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/rust-azure/212)
<!-- Reviewable:end -->
@bors-servo
Copy link

☀️ Test successful - travis

@Manishearth
Copy link
Member

@bors-servo r+

@bors-servo
Copy link

📌 Commit 0a0c2e2 has been approved by Manishearth

@bors-servo
Copy link

⌛ Testing commit 0a0c2e2 with merge 7662f94...

bors-servo pushed a commit that referenced this pull request Dec 14, 2015
Derive Debug for (almost) all public types

Leaving out DrawTarget/DrawTargetBacking, as the latter includes
GLRasterizationContext from Skia, which is missing Debug.

(And I don't feel like doing this for Skia right now -- this is already
quite getting out of hand...)

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/rust-azure/212)
<!-- Reviewable:end -->
@bors-servo
Copy link

☀️ Test successful - travis

@bors-servo bors-servo merged commit 0a0c2e2 into servo:master Dec 14, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants