From: Steve Holme <steve_holme@hotmail.com>
Date: Sun, 14 Dec 2014 11:45:14 +0000 (+0000)
Subject: ntlm: Remove unnecessary casts in readshort_le()
X-Git-Tag: curl-7_40_0~156
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=18f58c437f2607feac4e3c107e90f86691ac5d0a;p=curl

ntlm: Remove unnecessary casts in readshort_le()

I don't think both of my fix ups from yesterday were needed to fix the
compilation warning, so remove the one that I think is unnecessary and
let the next Android autobuild prove/disprove it.
---

diff --git a/lib/curl_ntlm_msgs.c b/lib/curl_ntlm_msgs.c
index 0998cb44c..d99a70f49 100644
--- a/lib/curl_ntlm_msgs.c
+++ b/lib/curl_ntlm_msgs.c
@@ -166,8 +166,8 @@ static unsigned int readint_le(unsigned char *buf)
  */
 static unsigned short readshort_le(unsigned char *buf)
 {
-  return (unsigned short)((unsigned short)((unsigned short)buf[0]) |
-                          (unsigned short)((unsigned short)buf[1] << 8));
+  return (unsigned short)(((unsigned short)buf[0]) |
+                          ((unsigned short)buf[1] << 8));
 }
 
 /*