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

Use new Vec#append_one() method #65

Merged
merged 3 commits into from
Apr 2, 2014

Conversation

drbawb
Copy link
Contributor

@drbawb drbawb commented Mar 31, 2014

  • The append_one() function was moved to a method of Vec.
  • Struct fields are now private by default.

See rust PR 13221 and rust PR 13184.

I've incorporated my visibility changes as I based them off my exp/append-one branch so I could continue building on my system. I've marked every field public which would've been public before the visibility rules changed.

The rust commit is only 5 hours old; so I'm not sure that Travis will build this yet.
(shake shake shake 🎱 Outlook Not So Good)

drbawb added 3 commits March 31, 2014 09:00
(Also added a type hint for the compiler to deduce what `flags_out`
contains at the time `append_one` is called.)
@AngryLawyer
Copy link
Member

Cheers :D

AngryLawyer added a commit that referenced this pull request Apr 2, 2014
Use new `Vec#append_one()` method
@AngryLawyer AngryLawyer merged commit d934eb3 into Rust-SDL2:master Apr 2, 2014
Cobrand pushed a commit that referenced this pull request Dec 2, 2016
Cobrand pushed a commit that referenced this pull request Dec 2, 2016
Cobrand pushed a commit that referenced this pull request Dec 2, 2016
fixes complaints about trying to include
sdl2-sys 0.24 and 0.25 in the same project.
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