Skip to content

Reduce warnings#145

Merged
ujh merged 8 commits intomasterfrom
reduce-warnings
Apr 17, 2015
Merged

Reduce warnings#145
ujh merged 8 commits intomasterfrom
reduce-warnings

Conversation

@ujh
Copy link
Owner

@ujh ujh commented Apr 17, 2015

cargo test now doesn't have any warnings anymore and cargo build only has two.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.49%) to 84.03% when pulling 5ce50c0 on reduce-warnings into c5144b9 on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.59%) to 83.92% when pulling c1ba864 on reduce-warnings into c5144b9 on master.

@ujh
Copy link
Owner Author

ujh commented Apr 17, 2015

Actually, now there's only one warning. It's for new_chain_liberties_greater_than, but I think you wanted to use that one @iopq, right?

@iopq
Copy link
Collaborator

iopq commented Apr 17, 2015

I will know if I use it or not when I actually implement the code for saving atari stones.

@ujh
Copy link
Owner Author

ujh commented Apr 17, 2015

OK, I'll leave it in then and you can remove it at some later point if you don't need it.

ujh added a commit that referenced this pull request Apr 17, 2015
@ujh ujh merged commit 2625ae7 into master Apr 17, 2015
@ujh ujh deleted the reduce-warnings branch April 17, 2015 08:59
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.

3 participants