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

Already on GitHub? Sign in to your account

Remove some warnings #12

Merged
merged 4 commits into from Jan 29, 2013

Conversation

Projects
None yet
2 participants
Contributor

kachick commented Jan 15, 2013

Please reject this commits if they were specified for readability. :)

Owner

aquasync commented Jan 29, 2013

Hi, sorry for the delay getting back to you. Looks pretty reasonable - made a few comments on the individual commits.

Contributor

kachick commented Jan 29, 2013

Thanks to your comments!
I have just tried some improvements for that.

@aquasync aquasync added a commit that referenced this pull request Jan 29, 2013

@aquasync aquasync Merge pull request #12 from kachick/improve-remove_warnings
Remove some warnings
913bb7f

@aquasync aquasync merged commit 913bb7f into aquasync:master Jan 29, 2013

Owner

aquasync commented Jan 29, 2013

Looks good. Thanks for your patience.

Contributor

kachick commented Jan 29, 2013

Thanks to merge! :)

@kachick kachick deleted the unknown repository branch Jan 29, 2013

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