Remove if checks after new.
authorLei Zhang <thestig@chromium.org>
Thu, 13 Aug 2015 22:24:19 +0000 (15:24 -0700)
committerLei Zhang <thestig@chromium.org>
Thu, 13 Aug 2015 22:24:19 +0000 (15:24 -0700)
R=tsepez@chromium.org

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

core/src/fxge/apple/fx_quartz_device.cpp
fpdfsdk/include/fxedit/fxet_edit.h
fpdfsdk/include/javascript/JS_Object.h
fpdfsdk/src/fxedit/fxet_list.cpp
fpdfsdk/src/javascript/app.cpp
fpdfsdk/src/pdfwindow/PWL_Wnd.cpp
samples/pdfium_test.cc

index 34851e0..61a0cef 100644 (file)
@@ -93,9 +93,6 @@ FX_BOOL CQuartz2D::drawGraphicsString(void* graphics,
   CGContextSaveGState(context);
 #if CGFLOAT_IS_DOUBLE
   CGPoint* glyphPositionsCG = new CGPoint[charsCount];
-  if (!glyphPositionsCG) {
-    return FALSE;
-  }
   for (int index = 0; index < charsCount; ++index) {
     glyphPositionsCG[index].x = glyphPositions[index].x;
     glyphPositionsCG[index].y = glyphPositions[index].y;
@@ -908,9 +905,6 @@ void CFX_QuartzDeviceDriver::setStrokeInfo(const CFX_GraphStateData* graphState,
   if (graphState->m_DashCount) {
 #if CGFLOAT_IS_DOUBLE
     CGFloat* dashArray = new CGFloat[graphState->m_DashCount];
-    if (!dashArray) {
-      return;
-    }
     for (int index = 0; index < graphState->m_DashCount; ++index) {
       dashArray[index] = graphState->m_DashArray[index];
     }
index d654ebc..2cb7c99 100644 (file)
@@ -115,8 +115,7 @@ class CFX_Edit_LineRectArray {
   }
 
   void Add(const CPVT_WordRange& wrLine, const CPDF_Rect& rcLine) {
-    if (CFX_Edit_LineRect* pRect = new CFX_Edit_LineRect(wrLine, rcLine))
-      m_LineRects.Add(pRect);
+    m_LineRects.Add(new CFX_Edit_LineRect(wrLine, rcLine));
   }
 
   int32_t GetSize() const { return m_LineRects.GetSize(); }
@@ -145,14 +144,14 @@ class CFX_Edit_RectArray {
   }
 
   void Add(const CPDF_Rect& rect) {
-    // check for overlaped area
-    for (int32_t i = 0, sz = m_Rects.GetSize(); i < sz; i++)
-      if (CPDF_Rect* pRect = m_Rects.GetAt(i))
-        if (pRect->Contains(rect))
-          return;
-
-    if (CPDF_Rect* pNewRect = new CPDF_Rect(rect))
-      m_Rects.Add(pNewRect);
+    // check for overlapped area
+    for (int32_t i = 0, sz = m_Rects.GetSize(); i < sz; i++) {
+      CPDF_Rect* pRect = m_Rects.GetAt(i);
+      if (pRect && pRect->Contains(rect))
+        return;
+    }
+
+    m_Rects.Add(new CPDF_Rect(rect));
   }
 
   int32_t GetSize() const { return m_Rects.GetSize(); }
index fd63d98..c0c3ab0 100644 (file)
@@ -109,11 +109,10 @@ struct JS_TIMER_MAPARRAY {
       if (JS_TIMER_MAP* pMap = m_Array.GetAt(i))
         pMap->pTimer = pTimer;
     } else {
-      if (JS_TIMER_MAP* pMap = new JS_TIMER_MAP) {
-        pMap->nID = nIndex;
-        pMap->pTimer = pTimer;
-        m_Array.Add(pMap);
-      }
+      JS_TIMER_MAP* pMap = new JS_TIMER_MAP;
+      pMap->nID = nIndex;
+      pMap->pTimer = pTimer;
+      m_Array.Add(pMap);
     }
   }
 
index 70dc8d5..ff4723e 100644 (file)
@@ -128,12 +128,11 @@ void CFX_List::SetFontSize(FX_FLOAT fFontSize) {
 }
 
 void CFX_List::AddItem(const FX_WCHAR* str) {
-  if (CFX_ListItem* pListItem = new CFX_ListItem()) {
-    pListItem->SetFontMap(m_pFontMap);
-    pListItem->SetFontSize(m_fFontSize);
-    pListItem->SetText(str);
-    m_aListItems.Add(pListItem);
-  }
+  CFX_ListItem* pListItem = new CFX_ListItem();
+  pListItem->SetFontMap(m_pFontMap);
+  pListItem->SetFontSize(m_fFontSize);
+  pListItem->SetText(str);
+  m_aListItems.Add(pListItem);
 }
 
 void CFX_List::ReArrange(int32_t nItemIndex) {
index 24f00ca..1b84508 100644 (file)
@@ -298,19 +298,12 @@ FX_BOOL app::alert(IFXJS_Context* cc,
       if (swMsg == L"") {
         CJS_Array carray(isolate);
         if (params[0].ConvertToArray(carray)) {
-          int iLenth = carray.GetLength();
+          int iLength = carray.GetLength();
           CJS_Value* pValue = new CJS_Value(isolate);
-          //                                   if (iLenth == 1)
-          //                                           pValue = new
-          // CJS_Value(isolate);
-          //                                   else if (iLenth > 1)
-          //                                           pValue = new
-          // CJS_Value[iLenth];
-
-          for (int i = 0; i < iLenth; i++) {
+          for (int i = 0; i < iLength; ++i) {
             carray.GetElement(i, *pValue);
             swMsg += (*pValue).ToCFXWideString();
-            if (i < iLenth - 1)
+            if (i < iLength - 1)
               swMsg += L",  ";
           }
 
index 81bbbbf..7b765f3 100644 (file)
@@ -685,8 +685,8 @@ void CPWL_Wnd::CreateVScrollBar(const PWL_CREATEPARAM& cp) {
     scp.eCursorType = FXCT_ARROW;
     scp.nTransparency = PWL_SCROLLBAR_TRANSPARANCY;
 
-    if ((m_pVScrollBar = new CPWL_ScrollBar(SBT_VSCROLL)))
-      m_pVScrollBar->Create(scp);
+    m_pVScrollBar = new CPWL_ScrollBar(SBT_VSCROLL);
+    m_pVScrollBar->Create(scp);
   }
 }
 
index 5a171c3..cb8fef3 100644 (file)
@@ -144,22 +144,20 @@ static void WritePpm(const char* pdf_name, int num, const void* buffer_void,
   // Source data is B, G, R, unused.
   // Dest data is R, G, B.
   char* result = new char[out_len];
-  if (result) {
-    for (int h = 0; h < height; ++h) {
-      const char* src_line = buffer + (stride * h);
-      char* dest_line = result + (width * h * 3);
-      for (int w = 0; w < width; ++w) {
-        // R
-        dest_line[w * 3] = src_line[(w * 4) + 2];
-        // G
-        dest_line[(w * 3) + 1] = src_line[(w * 4) + 1];
-        // B
-        dest_line[(w * 3) + 2] = src_line[w * 4];
-      }
+  for (int h = 0; h < height; ++h) {
+    const char* src_line = buffer + (stride * h);
+    char* dest_line = result + (width * h * 3);
+    for (int w = 0; w < width; ++w) {
+      // R
+      dest_line[w * 3] = src_line[(w * 4) + 2];
+      // G
+      dest_line[(w * 3) + 1] = src_line[(w * 4) + 1];
+      // B
+      dest_line[(w * 3) + 2] = src_line[w * 4];
     }
-    fwrite(result, out_len, 1, fp);
-    delete [] result;
   }
+  fwrite(result, out_len, 1, fp);
+  delete[] result;
   fclose(fp);
 }