Skip to content
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

Change CFMutableAttributedString::set_attribute argument type #370

Merged

Conversation

@brunogouveia
Copy link
Contributor

brunogouveia commented Apr 17, 2020

Type of the argument value can be a reference as opposed to requiring the value's ownership.

@jdm
Copy link
Member

jdm commented Apr 17, 2020

@bors-servo
Copy link
Contributor

bors-servo commented Apr 17, 2020

📌 Commit b68e9c7 has been approved by jdm

@bors-servo
Copy link
Contributor

bors-servo commented Apr 17, 2020

Testing commit b68e9c7 with merge 508a372...

@bors-servo
Copy link
Contributor

bors-servo commented Apr 17, 2020

☀️ Test successful - checks-travis
Approved by: jdm
Pushing 508a372 to master...

@bors-servo bors-servo merged commit 508a372 into servo:master Apr 17, 2020
2 checks passed
2 checks passed
Travis CI - Pull Request Build Passed
Details
homu Test successful
Details
@jrmuizel
Copy link
Collaborator

jrmuizel commented Jun 29, 2020

FWIW, this changed the api so it should've bumped the version number. See linebender/piet#220

@jdm
Copy link
Member

jdm commented Jun 29, 2020

I'm correcting that in #397.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.