Re-enable several MSVC warnings

core/fpdfapi/fpdf_render/fpdf_render_text.cpp

Index: core/fpdfapi/fpdf_render/fpdf_render_text.cpp
diff --git a/core/fpdfapi/fpdf_render/fpdf_render_text.cpp b/core/fpdfapi/fpdf_render/fpdf_render_text.cpp
index 4db224939d08422d784866c86f714fd71cfbdd69..a41d9be40a5e2e20e8d51588b8cbd69e3dfd6b65 100644
--- a/core/fpdfapi/fpdf_render/fpdf_render_text.cpp
+++ b/core/fpdfapi/fpdf_render/fpdf_render_text.cpp
@@ -135,7 +135,8 @@ CFX_GlyphBitmap* CPDF_Type3Cache::RenderGlyph(CPDF_Type3Glyphs* pSize,
   text_matrix.Set(pMatrix->a, pMatrix->b, pMatrix->c, pMatrix->d, 0, 0);
   image_matrix.Concat(text_matrix);
   CFX_DIBitmap* pResBitmap = NULL;
-  int left, top;
+  int left = 0;
+  int top = 0;
   if (FXSYS_fabs(image_matrix.b) < FXSYS_fabs(image_matrix.a) / 100 &&

[Tom Sepez, 2016/03/16 18:34:58]

Do we know if this is ever false? If we could somehow remove it, the
initializations wouldn't be required.  I can't see how it works otherwise.

[Wei Li, 2016/03/17 02:24:05]

Yes, it could be true or false. Tricky thing here is that |left| is passed by
reference as an output argument in line 170. 

[Tom Sepez, 2016/03/17 16:40:05]

Acknowledged.

       FXSYS_fabs(image_matrix.c) < FXSYS_fabs(image_matrix.d) / 100) {
     int top_line, bottom_line;

[Tom Sepez, 2016/03/16 18:34:58]

nit: one per line, combine with assignments.

[Wei Li, 2016/03/17 02:24:04]

Done.