diff options
author | Sinan Kaya <41809318+franksinankaya@users.noreply.github.com> | 2019-03-04 01:55:22 -0500 |
---|---|---|
committer | Jan Kotas <jkotas@microsoft.com> | 2019-03-03 22:55:22 -0800 |
commit | 9d3f264b9ef8b4715017ec615dcb6f9d57e607cc (patch) | |
tree | 44cdbdfc045ca93779e3d6faabe30ab2e4b56dd1 /src/ToolBox | |
parent | 7da0883d9ceba31bccf36cc8cbbbf2fe77a4f3e0 (diff) | |
download | coreclr-9d3f264b9ef8b4715017ec615dcb6f9d57e607cc.tar.gz coreclr-9d3f264b9ef8b4715017ec615dcb6f9d57e607cc.tar.bz2 coreclr-9d3f264b9ef8b4715017ec615dcb6f9d57e607cc.zip |
Cleanup more GCC warnings (#22872)
* Add parenthesis
src/vm/sha1.cpp: In function ‘void SHA1_block(SHA1_CTX*)’:
src/vm/sha1.cpp:93:29: warning: suggest parentheses around arithmetic in operand of ‘|’ [-Wparentheses]
#define ROUND3(B, C, D) ((C & (B | D) | (B & D)) + sha1_round3)
^
src/vm/sha1.cpp:139:32: note: in expansion of macro ‘ROUND3’
e += ROTATE32L(a, 5) + ROUND3(b, c, d) + msg80[i];
* Move declaration into same file as one was defined Extern the other one was static
* Remove hr=hr undefined assignment
* Fix mutli-line comment warning
* Convert multi-character literal
* Remove null check for stack local variables
rc/vm/invokeutil.cpp: In static member function ‘static void InvokeUtil::SetValidField(CorElementType, TypeHandle, FieldDesc*, OBJECTREF*, OBJECTREF*, TypeHandle, CLR_BOOL*)’:
src/vm/invokeutil.cpp:978:29: warning: the address of ‘Throwable’ will never be NULL [-Waddress]
EX_CATCH_THROWABLE(&Throwable);
^
src/inc/ex.h:1087:21: note: in definition of macro ‘EX_CATCH_THROWABLE’
if (NULL != ppThrowable)
^
Diffstat (limited to 'src/ToolBox')
-rw-r--r-- | src/ToolBox/SOS/Strike/sildasm.cpp | 1 | ||||
-rw-r--r-- | src/ToolBox/SOS/Strike/sos_md.h | 2 |
2 files changed, 1 insertions, 2 deletions
diff --git a/src/ToolBox/SOS/Strike/sildasm.cpp b/src/ToolBox/SOS/Strike/sildasm.cpp index 911ff091e5..097376d553 100644 --- a/src/ToolBox/SOS/Strike/sildasm.cpp +++ b/src/ToolBox/SOS/Strike/sildasm.cpp @@ -65,6 +65,7 @@ static OpCode opcodes[] = static ULONG position = 0; static BYTE *pBuffer = NULL; +static char* asString(CQuickBytes *out); // The UNALIGNED is because on IA64 alignment rules would prevent // us from reading a pointer from an unaligned source. diff --git a/src/ToolBox/SOS/Strike/sos_md.h b/src/ToolBox/SOS/Strike/sos_md.h index b589109986..a7cd30c3bc 100644 --- a/src/ToolBox/SOS/Strike/sos_md.h +++ b/src/ToolBox/SOS/Strike/sos_md.h @@ -872,8 +872,6 @@ typedef enum } PPFormatFlags; -char* asString(CQuickBytes *out); - PCCOR_SIGNATURE PrettyPrintType( PCCOR_SIGNATURE typePtr, // type to convert, CQuickBytes *out, // where to put the pretty printed string |