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

Modulo operators for sf::Time (fixes #429) #430

Merged
merged 2 commits into from Aug 12, 2013

Conversation

Projects
None yet
3 participants
@Foaly
Contributor

Foaly commented Jul 17, 2013

Since I already had it implemented in my copy of SFML, I made a clean branch and send a pull request, to save everybody some work. It's tested and it works.

@ghost ghost assigned LaurentGomila Jul 17, 2013

@LaurentGomila

This comment has been minimized.

Show comment
Hide comment
@LaurentGomila

LaurentGomila Aug 11, 2013

Member

There should be 2 empty lines between function definitions (in .cpp). Please fix it, I'd like to merge your pull request :p

Member

LaurentGomila commented Aug 11, 2013

There should be 2 empty lines between function definitions (in .cpp). Please fix it, I'd like to merge your pull request :p

@Foaly

This comment has been minimized.

Show comment
Hide comment
@Foaly

Foaly Aug 11, 2013

Contributor

Sounds great! I changed it.

Contributor

Foaly commented Aug 11, 2013

Sounds great! I changed it.

LaurentGomila added a commit that referenced this pull request Aug 12, 2013

Merge pull request #430 from Foaly/timeOperators
Added modulo operators for sf::Time (#429)

@LaurentGomila LaurentGomila merged commit a7f7c57 into SFML:master Aug 12, 2013

@Foaly Foaly deleted the Foaly:timeOperators branch Aug 19, 2013

@Foaly Foaly restored the Foaly:timeOperators branch Apr 27, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment