Skip to content

/ core-foundation-rs

New issue

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

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

Already on GitHub? Sign in to your account

Use full path to mem::transmute in macro #74

Merged
merged 1 commit into from Dec 17, 2015
Merged

Use full path to mem::transmute in macro #74

merged 1 commit into from Dec 17, 2015
+2 −8

Conversation

@TimNN
Copy link
Contributor

TimNN commented Dec 17, 2015

Also removes now unused std::mem uses.

Review on Reviewable
@TimNN
Use full path to mem::transmute in macro
Loading status checks…
d2e9195
@metajack
Copy link
Contributor

metajack commented Dec 17, 2015

@bors-servo r+

Reviewed 7 of 7 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io
@bors-servo
Copy link
Contributor

bors-servo commented Dec 17, 2015

📌 Commit d2e9195 has been approved by metajack
@bors-servo
Copy link
Contributor

bors-servo commented Dec 17, 2015

⌛ Testing commit d2e9195 with merge edfb681...
bors-servo added a commit that referenced this pull request Dec 17, 2015
@bors-servo
Auto merge of #74 - TimNN:macro-full-path, r=metajack
Loading status checks…
edfb681 
Use full path to mem::transmute in macro

Also removes now unused `std::mem` uses.

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

bors-servo commented Dec 17, 2015

☀️ Test successful - travis
@bors-servo bors-servo merged commit d2e9195 into servo:master Dec 17, 2015
3 checks passed
3 checks passed
code-review/reviewable Review complete: all files reviewed, all discussions resolved
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@TimNN TimNN deleted the TimNN:macro-full-path branch Dec 17, 2015
@frewsxcv frewsxcv mentioned this pull request Apr 20, 2016
Merged
bors-servo added a commit that referenced this pull request Apr 20, 2016
@bors-servo
Auto merge of #78 - servo:bump, r=mbrubeck
Loading status checks…
c3c9978 
Bump version: 0.2.0 -> 0.2.1.

#77

#75

#73

#74

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/core-foundation-rs/78)
<!-- Reviewable:end -->
jdm pushed a commit that referenced this pull request Feb 1, 2018
@brendanzab
Merge pull request #74 from pcwalton/fpret
bb12c9c 
Use `objc_msgSend_fpret` for `backingStoreFactor`.
jdm pushed a commit that referenced this pull request Feb 1, 2018
@bors-servo
Auto merge of #74 - msanders:ms/set-event-flags, r=jdm
43f708c 
Allow applying multiple bitmask flags to events

<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/core-graphics-rs/74)
<!-- Reviewable:end -->
jdm pushed a commit that referenced this pull request Feb 1, 2018
@bors-servo
Auto merge of #74 - nox:bump, r=jdm
a920837 
Update core-graphics to 0.12.1

<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/core-text-rs/74)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
@TimNN @metajack @bors-servo
You can’t perform that action at this time.