fix 2 compiler warnings #1171

Merged
merged 2 commits into from May 1, 2012

Conversation

Projects
None yet
3 participants
@Diapolo

Diapolo commented Apr 30, 2012

No description provided.

@jgarzik

This comment has been minimized.

Show comment
Hide comment
@jgarzik

jgarzik May 1, 2012

Contributor

Please add the specific warning being fixed to the commit messages. "fix compiler warning" is vague, and will get even more vague over time. Code changes look OK

Contributor

jgarzik commented May 1, 2012

Please add the specific warning being fixed to the commit messages. "fix compiler warning" is vague, and will get even more vague over time. Code changes look OK

@sipa

This comment has been minimized.

Show comment
Hide comment
@sipa

sipa May 1, 2012

Member

ACK

Member

sipa commented May 1, 2012

ACK

@jgarzik

This comment has been minimized.

Show comment
Hide comment
@jgarzik

jgarzik May 1, 2012

Contributor

Sorry, to be more specific, please redo the commits to include the warning being fixed inside the git commit itself. Need to get that useful info into the permanent git history.

Contributor

jgarzik commented May 1, 2012

Sorry, to be more specific, please redo the commits to include the warning being fixed inside the git commit itself. Need to get that useful info into the permanent git history.

Philip Kaufmann added some commits Apr 30, 2012

@Diapolo

This comment has been minimized.

Show comment
Hide comment
@Diapolo

Diapolo May 1, 2012

Renamed the commit-messages to be more detailed.

Diapolo commented May 1, 2012

Renamed the commit-messages to be more detailed.

@jgarzik

This comment has been minimized.

Show comment
Hide comment
@jgarzik

jgarzik May 1, 2012

Contributor

ACK

Contributor

jgarzik commented May 1, 2012

ACK

jgarzik pushed a commit that referenced this pull request May 1, 2012

@jgarzik jgarzik merged commit ca97bde into bitcoin:master May 1, 2012

coblee pushed a commit to litecoin-project/litecoin that referenced this pull request Jul 17, 2012

suprnurd pushed a commit to chaincoin-legacy/chaincoin that referenced this pull request Dec 5, 2017

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