Cleanup: Fix some unused-function warnings.
authorLei Zhang <thestig@chromium.org>
Sat, 7 Feb 2015 00:51:37 +0000 (16:51 -0800)
committerLei Zhang <thestig@chromium.org>
Sat, 7 Feb 2015 00:51:37 +0000 (16:51 -0800)
BUG=pdfium:29
R=tsepez@chromium.org

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

core/src/fpdfapi/fpdf_basic_module.cpp
core/src/fpdfapi/fpdf_render/fpdf_render_loadimage.cpp
core/src/fpdfdoc/doc_basic.cpp
core/src/fxge/ge/fx_ge_fontmap.cpp
fpdfsdk/src/fxedit/fxet_pageobjs.cpp

index edccda5..82658ab 100644 (file)
@@ -61,23 +61,6 @@ FX_BOOL CPDF_ModuleMgr::DownloadModule(FX_LPCSTR module_name)
     }
     return m_pDownloadCallback(module_name);
 }
-static CFX_ByteString _GetPath(const CFX_ByteString& folder, FX_LPCSTR name)
-{
-    FX_STRSIZE folder_len = folder.GetLength();
-#if _FX_OS_ == _FX_SYMBIAN_ || _FXM_PLATFORM_  == _FXM_PLATFORM_WINDOWS_
-    if (folder[folder_len - 1] == '\\') {
-        return folder + name;
-    } else {
-        return (folder + "\\") + name;
-    }
-#else
-    if (folder[folder_len - 1] == '/') {
-        return folder + name;
-    } else {
-        return (folder + "/") + name;
-    }
-#endif
-}
 void CPDF_ModuleMgr::NotifyModuleAvailable(FX_LPCSTR module_name)
 {
     if (FXSYS_strcmp(module_name, ADDIN_NAME_CJK) == 0) {
index 45d0078..4583bc1 100644 (file)
@@ -329,7 +329,6 @@ int CPDF_DIBSource::StartLoadDIBSource(CPDF_Document* pDoc, const CPDF_Stream* p
     if (m_pStreamAcc->GetSize() == 0 || m_pStreamAcc->GetData() == NULL) {
         return 0;
     }
-    const CFX_ByteString& decoder = m_pStreamAcc->GetImageDecoder();
     int ret = CreateDecoder();
     if (ret != 1) {
         if (!ret) {
index 459ae87..496ed86 100644 (file)
@@ -250,6 +250,7 @@ CPDF_Array* CPDF_NameTree::LookupNamedDest(CPDF_Document* pDoc, FX_BSTR sName)
     }
     return NULL;
 }
+#if _FXM_PLATFORM_  == _FXM_PLATFORM_APPLE_ || _FXM_PLATFORM_  == _FXM_PLATFORM_WINDOWS_
 static CFX_WideString ChangeSlashToPlatform(FX_LPCWSTR str)
 {
     CFX_WideString result;
@@ -257,10 +258,8 @@ static CFX_WideString ChangeSlashToPlatform(FX_LPCWSTR str)
         if (*str == '/') {
 #if _FXM_PLATFORM_  == _FXM_PLATFORM_APPLE_
             result += ':';
-#elif _FXM_PLATFORM_  == _FXM_PLATFORM_WINDOWS_
-            result += '\\';
 #else
-            result += *str;
+            result += '\\';
 #endif
         } else {
             result += *str;
@@ -269,6 +268,20 @@ static CFX_WideString ChangeSlashToPlatform(FX_LPCWSTR str)
     }
     return result;
 }
+static CFX_WideString ChangeSlashToPDF(FX_LPCWSTR str)
+{
+    CFX_WideString result;
+    while (*str) {
+        if (*str == '\\' || *str == ':') {
+            result += '/';
+        } else {
+            result += *str;
+        }
+        str++;
+    }
+    return result;
+}
+#endif
 static CFX_WideString FILESPEC_DecodeFileName(FX_WSTR filepath)
 {
     if (filepath.GetLength() <= 1) {
@@ -349,19 +362,6 @@ FX_BOOL CPDF_FileSpec::IsURL() const
     }
     return ((CPDF_Dictionary*)m_pObj)->GetString(FX_BSTRC("FS")) == FX_BSTRC("URL");
 }
-static CFX_WideString ChangeSlashToPDF(FX_LPCWSTR str)
-{
-    CFX_WideString result;
-    while (*str) {
-        if (*str == '\\' || *str == ':') {
-            result += '/';
-        } else {
-            result += *str;
-        }
-        str++;
-    }
-    return result;
-}
 CFX_WideString FILESPEC_EncodeFileName(FX_WSTR filepath)
 {
     if (filepath.GetLength() <= 1) {
index 22c3991..079c485 100644 (file)
@@ -748,19 +748,6 @@ FX_DWORD _GetCodePageRangeFromCharset(int charset)
     }
     return 1 << 21;
 }
-static int CP2CharSet(int cp)
-{
-    if(cp == 932) {
-        return FXFONT_SHIFTJIS_CHARSET;
-    } else if(cp == 936) {
-        return FXFONT_GB2312_CHARSET;
-    } else if(cp == 949) {
-        return FXFONT_HANGEUL_CHARSET;
-    } else if(cp == 950) {
-        return FXFONT_CHINESEBIG5_CHARSET;
-    }
-    return FXFONT_DEFAULT_CHARSET;
-}
 FXFT_Face CFX_FontMapper::UseInternalSubst(CFX_SubstFont* pSubstFont, int iBaseFont, int italic_angle, int weight, int picthfamily)
 {
     if (iBaseFont < 12) {
index e5cbf4e..7b64b33 100644 (file)
@@ -397,28 +397,6 @@ void IFX_Edit::DrawRichEdit(CFX_RenderDevice* pDevice, CPDF_Matrix* pUser2Device
        pDevice->RestoreState();
 }
 
-static void AddLineToPageObjects(CPDF_PageObjects* pPageObjs, FX_COLORREF crStroke, 
-                                                                const CPDF_Point& pt1, const CPDF_Point& pt2)
-{
-       CPDF_PathObject* pPathObj = new CPDF_PathObject;
-       CPDF_PathData* pPathData = pPathObj->m_Path.GetModify();
-
-       pPathData->SetPointCount(2);
-       pPathData->SetPoint(0, pt1.x, pt1.y, FXPT_MOVETO);
-       pPathData->SetPoint(1, pt2.x, pt2.y, FXPT_LINETO);
-
-       FX_FLOAT rgb[3];
-       rgb[0] = FXARGB_R(crStroke) / 255.0f;
-       rgb[1] = FXARGB_G(crStroke) / 255.0f;
-       rgb[2] = FXARGB_B(crStroke) / 255.0f;
-       pPathObj->m_ColorState.SetStrokeColor(CPDF_ColorSpace::GetStockCS(PDFCS_DEVICERGB), rgb, 3);
-
-       CFX_GraphStateData* pData = pPathObj->m_GraphState.GetModify();
-       pData->m_LineWidth = 1;
-
-       pPageObjs->InsertObject(pPageObjs->GetLastObjectPosition(),pPathObj);
-}
-
 static void AddRectToPageObjects(CPDF_PageObjects* pPageObjs, FX_COLORREF crFill, const CPDF_Rect& rcFill)
 {
        CPDF_PathObject* pPathObj = new CPDF_PathObject;