Removing unnecessary casts from wchar_t* to wchar_t*, by various names.
authorBruce Dawson <brucedawson@google.com>
Mon, 17 Nov 2014 23:33:04 +0000 (15:33 -0800)
committerBruce Dawson <brucedawson@google.com>
Mon, 17 Nov 2014 23:33:04 +0000 (15:33 -0800)
Remove casts that merely cast from wchar_t* to wchar_t*. Sometimes the
types or casts are FX_LPCWSTR but the idea is the same. Excess casts
can (and have) hidden bugs so removing these may prevent future problems.

Original patch from Bruce Dawson(brucedawson@chromium.org)

R=bo_xu@foxitsoftware.com, tsepez@chromium.org

Review URL: https://codereview.chromium.org/730993002

core/include/fxcrt/fx_string.h
core/src/fxcrt/fx_basic_wstring.cpp
fpdfsdk/include/javascript/JS_Define.h
fpdfsdk/src/javascript/Field.cpp
fpdfsdk/src/javascript/global.cpp
fpdfsdk/src/javascript/util.cpp
fpdfsdk/src/jsapi/fxjs_v8.cpp

index 26b04b7..364c510 100644 (file)
@@ -597,7 +597,7 @@ private:
     }
 };
 typedef const CFX_WideStringC& FX_WSTR;
-#define FX_WSTRC(wstr) CFX_WideStringC((FX_LPCWSTR)wstr, sizeof(wstr) / sizeof(FX_WCHAR) - 1)
+#define FX_WSTRC(wstr) CFX_WideStringC(wstr, sizeof(wstr) / sizeof(FX_WCHAR) - 1)
 struct CFX_StringDataW {
 
     long               m_nRefs;
index 3a13d59..0827fb6 100644 (file)
@@ -586,7 +586,7 @@ FX_STRSIZE CFX_WideString::Find(FX_LPCWSTR lpszSub, FX_STRSIZE nStart) const
     if (nLength < 1 || nStart > nLength) {
         return -1;
     }
-    FX_LPCWSTR lpsz = (FX_LPCWSTR)FXSYS_wcsstr(m_pData->m_String + nStart, lpszSub);
+    FX_LPCWSTR lpsz = FXSYS_wcsstr(m_pData->m_String + nStart, lpszSub);
     return (lpsz == NULL) ? -1 : (int)(lpsz - m_pData->m_String);
 }
 FX_STRSIZE CFX_WideString::Find(FX_WCHAR ch, FX_STRSIZE nStart) const
@@ -598,7 +598,7 @@ FX_STRSIZE CFX_WideString::Find(FX_WCHAR ch, FX_STRSIZE nStart) const
     if (nStart >= nLength) {
         return -1;
     }
-    FX_LPCWSTR lpsz = (FX_LPCWSTR)FXSYS_wcschr(m_pData->m_String + nStart, ch);
+    FX_LPCWSTR lpsz = FXSYS_wcschr(m_pData->m_String + nStart, ch);
     return (lpsz == NULL) ? -1 : (int)(lpsz - m_pData->m_String);
 }
 void CFX_WideString::TrimRight(FX_LPCWSTR lpszTargetList)
index 108c1e2..62a8d14 100644 (file)
@@ -626,7 +626,7 @@ if (JS_DefineGlobalConst(pRuntime,JS_WIDESTRING(const_name),JS_NewString(pRuntim
 int size = sizeof(ArrayContent) / sizeof(FX_LPCWSTR);\
 \
 CJS_Array array(pRuntime);\
-for (int i=0; i<size; i++) array.SetElement(i,CJS_Value(pRuntime,(FX_LPCWSTR)ArrayContent[i]));\
+for (int i=0; i<size; i++) array.SetElement(i,CJS_Value(pRuntime,ArrayContent[i]));\
 \
 CJS_PropValue prop(pRuntime);\
 prop << array;\
index 73a4e0a..a845992 100644 (file)
@@ -149,7 +149,7 @@ void Field::ParseFieldName(const std::wstring &strFieldNameParsed,std::wstring &
                return;
        }
        std::wstring suffixal = strFieldNameParsed.substr(iStart+1);
-       iControlNo = FXSYS_wtoi((FX_LPCWSTR)suffixal.c_str());
+       iControlNo = FXSYS_wtoi(suffixal.c_str());
        if (iControlNo == 0)
        {
                int iStart;
@@ -194,7 +194,7 @@ FX_BOOL Field::AttachField(Document* pDocument, const CFX_WideString& csFieldNam
        {
                std::wstring strFieldName;
                int iControlNo = -1;
-               ParseFieldName((wchar_t*)(FX_LPCWSTR)swFieldNameTemp, strFieldName, iControlNo);
+               ParseFieldName((FX_LPCWSTR)swFieldNameTemp, strFieldName, iControlNo);
                if (iControlNo == -1) return FALSE;
                
                m_FieldName = strFieldName.c_str();
index 295a8d4..03bdd5b 100644 (file)
@@ -341,7 +341,7 @@ void global_alternate::ObjectToArray(v8::Handle<v8::Object> pObj, CJS_GlobalVari
                CFX_WideString ws = JS_ToString(JS_GetArrayElemnet(pKeyList, i));
                CFX_ByteString sKey = ws.UTF8Encode();
 
-               v8::Handle<v8::Value> v = JS_GetObjectElement(isolate, pObj, (const wchar_t*)(FX_LPCWSTR)ws);
+               v8::Handle<v8::Value> v = JS_GetObjectElement(isolate, pObj, (FX_LPCWSTR)ws);
                FXJSVALUETYPE vt = GET_VALUE_TYPE(v);
                switch (vt)
                {
index 1e5c751..d2813f5 100644 (file)
@@ -182,23 +182,23 @@ FX_BOOL util::printf(OBJ_METHOD_PARAMS)
                switch (ParstDataType(&c_strFormat))
                {
                        case UTIL_INT:
-                               strSegment.Format((FX_LPCWSTR)c_strFormat.c_str(),(int)params[iIndex]);
+                               strSegment.Format(c_strFormat.c_str(),(int)params[iIndex]);
                                break;
                        case UTIL_DOUBLE:
-                               strSegment.Format((FX_LPCWSTR)c_strFormat.c_str(),(double)params[iIndex]);
+                               strSegment.Format(c_strFormat.c_str(),(double)params[iIndex]);
                                break;
                        case UTIL_STRING:
-                               strSegment.Format((FX_LPCWSTR)c_strFormat.c_str(),(FX_LPCWSTR)params[iIndex].operator CFX_WideString());
+                               strSegment.Format(c_strFormat.c_str(),(FX_LPCWSTR)params[iIndex].operator CFX_WideString());
                                break;
                        default:
-                               strSegment.Format(L"%S", (FX_LPCWSTR)c_strFormat.c_str());
+                               strSegment.Format(L"%S", c_strFormat.c_str());
                                break;
                }
-               c_strResult += (wchar_t*)strSegment.GetBuffer(strSegment.GetLength()+1);
+               c_strResult += strSegment.GetBuffer(strSegment.GetLength()+1);
        }
 
        c_strResult.erase(c_strResult.begin());
-       vRet = (FX_LPCWSTR)c_strResult.c_str();
+       vRet = c_strResult.c_str();
        return TRUE;
 }
 
@@ -271,7 +271,7 @@ FX_BOOL util::printd(OBJ_METHOD_PARAMS)
        }
        else if (p1.GetType() == VT_string)
        {
-               std::basic_string<wchar_t> cFormat = (wchar_t*)(FX_LPCWSTR)p1.operator CFX_WideString();                
+               std::basic_string<wchar_t> cFormat = (FX_LPCWSTR)p1.operator CFX_WideString();          
 
                bool bXFAPicture = false;
                if (iSize > 2)
@@ -365,7 +365,7 @@ FX_BOOL util::printd(OBJ_METHOD_PARAMS)
                wchar_t buf[64] = {0};
                strFormat = wcsftime(buf, 64, cFormat.c_str(), &time);
                cFormat = buf;
-               vRet = (FX_LPCWSTR)cFormat.c_str();
+               vRet = cFormat.c_str();
                //rtRet = strFormat.GetBuffer(strFormat.GetLength()+1);
                return TRUE;
        }
index 5eb9873..7d1226e 100644 (file)
@@ -93,7 +93,7 @@ int JS_DefineObjMethod(IJS_Runtime* pJSRuntime, int nObjDefnID, const wchar_t* s
        v8::Isolate::Scope isolate_scope(isolate);
        v8::HandleScope handle_scope(isolate);
 
-       CFX_WideString ws = CFX_WideString((FX_LPCWSTR)sMethodName);
+       CFX_WideString ws = CFX_WideString(sMethodName);
        CFX_ByteString bsMethodName = ws.UTF8Encode();
 
        CFX_PtrArray* pArray = (CFX_PtrArray*)isolate->GetData(0);
@@ -113,7 +113,7 @@ int JS_DefineObjProperty(IJS_Runtime* pJSRuntime, int nObjDefnID, const wchar_t*
        v8::Isolate::Scope isolate_scope(isolate);
        v8::HandleScope handle_scope(isolate);
 
-       CFX_WideString ws = CFX_WideString((FX_LPCWSTR)sPropName);
+       CFX_WideString ws = CFX_WideString(sPropName);
        CFX_ByteString bsPropertyName = ws.UTF8Encode();
 
        CFX_PtrArray* pArray = (CFX_PtrArray*)isolate->GetData(0);
@@ -153,7 +153,7 @@ int JS_DefineObjConst(IJS_Runtime* pJSRuntime, int nObjDefnID, const wchar_t* sC
        CFX_PtrArray* pArray = (CFX_PtrArray*)isolate->GetData(0);
        if(!pArray) return 0;
 
-       CFX_WideString ws = CFX_WideString((FX_LPCWSTR)sConstName);
+       CFX_WideString ws = CFX_WideString(sConstName);
        CFX_ByteString bsConstName = ws.UTF8Encode();
 
        if(nObjDefnID<0 || nObjDefnID>= pArray->GetSize()) return 0;
@@ -188,7 +188,7 @@ int JS_DefineGlobalMethod(IJS_Runtime* pJSRuntime, const wchar_t* sMethodName, v
        v8::Isolate::Scope isolate_scope(isolate);
        v8::HandleScope handle_scope(isolate);
 
-       CFX_WideString ws = CFX_WideString((FX_LPCWSTR)sMethodName);
+       CFX_WideString ws = CFX_WideString(sMethodName);
        CFX_ByteString bsMethodName = ws.UTF8Encode();
 
        v8::Local<v8::FunctionTemplate> funTempl = v8::FunctionTemplate::New(isolate, pMethodCall);
@@ -212,7 +212,7 @@ int JS_DefineGlobalConst(IJS_Runtime* pJSRuntime, const wchar_t* sConstName, v8:
        v8::Isolate::Scope isolate_scope(isolate);
        v8::HandleScope handle_scope(isolate);
 
-       CFX_WideString ws = CFX_WideString((FX_LPCWSTR)sConstName);
+       CFX_WideString ws = CFX_WideString(sConstName);
        CFX_ByteString bsConst= ws.UTF8Encode();
 
        v8::Local<v8::ObjectTemplate> objTemp;