Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2787)

Unified Diff: core/src/fpdfapi/fpdf_page/fpdf_page_func.cpp

Issue 1148353002: Fix four annoying warnings: (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: Fix check. Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: core/src/fpdfapi/fpdf_page/fpdf_page_func.cpp
diff --git a/core/src/fpdfapi/fpdf_page/fpdf_page_func.cpp b/core/src/fpdfapi/fpdf_page/fpdf_page_func.cpp
index 8ec490a9e87a40f95be7080b067cfe96cea27569..2094fbbda6995cd9d15634ede643b63a017bf883 100644
--- a/core/src/fpdfapi/fpdf_page/fpdf_page_func.cpp
+++ b/core/src/fpdfapi/fpdf_page/fpdf_page_func.cpp
@@ -73,7 +73,9 @@ CPDF_PSProc::~CPDF_PSProc()
}
}
}
+#if _FXM_PLATFORM_ == _FX_PLATFORM_WINDOWS_
Lei Zhang 2015/05/20 21:43:16 Hans: do you know if Clang for Windows will suppor
hans 2015/05/20 21:53:07 It doesn't support it currently, and I don't see i
Tom Sepez 2015/05/21 00:07:23 Remove it and HFTB.
Lei Zhang 2015/05/21 00:14:08 Do you want to ask Jun about this? He may know why
#pragma optimize( "", off )
+#endif
FX_BOOL CPDF_PSProc::Execute(CPDF_PSEngine* pEngine)
{
int size = m_Operators.GetSize();
@@ -107,7 +109,9 @@ FX_BOOL CPDF_PSProc::Execute(CPDF_PSEngine* pEngine)
}
return TRUE;
}
+#if _FXM_PLATFORM_ == _FXM_PLATFORM_WINDOWS_
#pragma optimize( "", on )
+#endif
CPDF_PSEngine::CPDF_PSEngine()
{
m_StackCount = 0;

Powered by Google App Engine
This is Rietveld 408576698