Skip to content

Reenable C4244 in repo #70026

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.

Sign up for GitHub

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

Merged
AaronRobinsonMSFT merged 20 commits into from
Jun 14, 2022
Merged

Reenable C4244 in repo #70026

Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
6c29b7c
Reenable 4244 warning for Mono.
AaronRobinsonMSFT May 31, 2022
2e6f9d1
Enable compiler errors for gcc/clang.
AaronRobinsonMSFT Jun 1, 2022
adcceba
Fix issues enabling -Werror=implicit-int-conversion
AaronRobinsonMSFT Jun 1, 2022
3406b91
More violations
AaronRobinsonMSFT Jun 1, 2022
0bd86a2
More violations
AaronRobinsonMSFT Jun 1, 2022
1915a34
More violations
AaronRobinsonMSFT Jun 1, 2022
e753237
More violations
AaronRobinsonMSFT Jun 1, 2022
9016821
More violations
AaronRobinsonMSFT Jun 3, 2022
e776f5f
More violations
AaronRobinsonMSFT Jun 6, 2022
2ee634b
More violations
AaronRobinsonMSFT Jun 7, 2022
608d3e5
More violations
AaronRobinsonMSFT Jun 7, 2022
4013d1a
More violations
AaronRobinsonMSFT Jun 7, 2022
7a9e1ff
More violations
AaronRobinsonMSFT Jun 7, 2022
4030b47
More violations
AaronRobinsonMSFT Jun 8, 2022
7ae1382
More violations
AaronRobinsonMSFT Jun 8, 2022
69c5467
More violations
AaronRobinsonMSFT Jun 8, 2022
4d64bdb
More violations
AaronRobinsonMSFT Jun 8, 2022
b3ddeae
More violations
AaronRobinsonMSFT Jun 8, 2022
4b83ea8
Revert turning on implicit-int-conversion as error.
AaronRobinsonMSFT Jun 9, 2022
54fae18
Review feedback
AaronRobinsonMSFT Jun 13, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
More violations
  • Loading branch information
@AaronRobinsonMSFT
AaronRobinsonMSFT committed Jun 8, 2022
commit 7ae138248176081ea72808dfee823dd923420c07
4 changes: 2 additions & 2 deletions src/coreclr/pal/src/safecrt/input.inl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -867,7 +867,7 @@ getnum:

if (_ISXDIGIT(ch)) {
num64 <<= 4;
ch = _hextodec(ch);
ch = _hextodec((_TCHAR)ch);
}
else
++done_flag;
Expand Down Expand Up @@ -910,7 +910,7 @@ getnum:

if (_ISXDIGIT(ch)) {
number = (number << 4);
ch = _hextodec(ch);
ch = _hextodec((_TCHAR)ch);
}
else
++done_flag;
Expand Down