diff options
author | Sergey Andreenko <seandree@microsoft.com> | 2018-06-14 18:32:41 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-06-14 18:32:41 -0700 |
commit | 1c8c96ec2bb52368124928648d7557ac85ff9c40 (patch) | |
tree | b3e231f27232ea8532f1914f23f982914df23836 /src/vm/methodtablebuilder.cpp | |
parent | 793daddd90caa014dc2dacebd6b73035ae57bae5 (diff) | |
download | coreclr-1c8c96ec2bb52368124928648d7557ac85ff9c40.tar.gz coreclr-1c8c96ec2bb52368124928648d7557ac85ff9c40.tar.bz2 coreclr-1c8c96ec2bb52368124928648d7557ac85ff9c40.zip |
clean up list of disabled warnings. (#18318)
* delete warnings that do not longer exist
For example C4171 was deleted after VS 6.0 (https://docs.microsoft.com/en-us/previous-versions/visualstudio/visual-studio-6.0/aa233011(v=vs.60))
* delete C4206 fromm the list
because its default value is 4, so this line is useless.
* reenable warning as error.
* enable warning C4430
and fix places that trigger it.
* fix C4334
* format the list
* fix ssize_t
Diffstat (limited to 'src/vm/methodtablebuilder.cpp')
-rw-r--r-- | src/vm/methodtablebuilder.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/vm/methodtablebuilder.cpp b/src/vm/methodtablebuilder.cpp index 0c1302eda1..bbed188b62 100644 --- a/src/vm/methodtablebuilder.cpp +++ b/src/vm/methodtablebuilder.cpp @@ -7924,7 +7924,7 @@ VOID MethodTableBuilder::PlaceInstanceFields(MethodTable ** pByValueClassCach for (i = 0; i < MAX_LOG2_PRIMITIVE_FIELD_SIZE; i++) { DWORD j; - if (IS_ALIGNED(dwCumulativeInstanceFieldPos, 1<<(i+1))) + if (IS_ALIGNED(dwCumulativeInstanceFieldPos, size_t{ 1 } << (i + 1))) continue; // check whether there are any bigger fields @@ -7976,7 +7976,7 @@ VOID MethodTableBuilder::PlaceInstanceFields(MethodTable ** pByValueClassCach } // Place the field - dwCumulativeInstanceFieldPos = (DWORD)ALIGN_UP(dwCumulativeInstanceFieldPos, 1 << i); + dwCumulativeInstanceFieldPos = (DWORD)ALIGN_UP(dwCumulativeInstanceFieldPos, size_t{ 1 } << i); pFieldDescList[j].SetOffset(dwCumulativeInstanceFieldPos - dwOffsetBias); dwCumulativeInstanceFieldPos += (1 << i); |