Skip to content

Fix MSVC warning.#935

Merged
LebedevRI merged 1 commit intogoogle:masterfrom
ben-clayton:fix-msvc-warning
Feb 7, 2020
Merged

Fix MSVC warning.#935
LebedevRI merged 1 commit intogoogle:masterfrom
ben-clayton:fix-msvc-warning

Conversation

@ben-clayton
Copy link
Copy Markdown
Contributor

This fixes the Visual Studio 2019 warning:

C4244: '=': conversion from 'int' to 'char', possible loss of data

When implicitly casting the return value of tolower() (int) to char.

Fixes: #932

@dmah42 dmah42 self-requested a review February 7, 2020 17:44
Copy link
Copy Markdown
Member

@dmah42 dmah42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM pending CI runs.

This fixes the Visual Studio 2019 warning:

`C4244: '=': conversion from 'int' to 'char', possible loss of data`

When implicitly casting the return value of tolower() (int) to char.

Fixes: google#932
@coveralls
Copy link
Copy Markdown

coveralls commented Feb 7, 2020

Coverage Status

Coverage remained the same at 92.041% when pulling b1e89fd on ben-clayton:fix-msvc-warning into e5ea03c on google:master.

@ben-clayton
Copy link
Copy Markdown
Contributor Author

continuous-integration/travis-ci/pr failure looks like a flake.

@LebedevRI LebedevRI merged commit 8982e1e into google:master Feb 7, 2020
@LebedevRI
Copy link
Copy Markdown
Collaborator

Thank you for the contribution!

@ben-clayton ben-clayton deleted the fix-msvc-warning branch February 7, 2020 23:51
JBakamovic pushed a commit to JBakamovic/benchmark that referenced this pull request Sep 11, 2020
This fixes the Visual Studio 2019 warning:

`C4244: '=': conversion from 'int' to 'char', possible loss of data`

When implicitly casting the return value of tolower() (int) to char.

Fixes: google#932
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Warning compiling benchmark with VS 2019

5 participants