commit d71f7b3e43e3a47493da434e84c2e92ed36d1aab
author Lei Zhang <thestig@chromium.org> Mon Jun 22 17:15:59 2020 +0000
committer Chromium commit bot <commit-bot@chromium.org> Mon Jun 22 17:15:59 2020 +0000
tree 084a1048a647c321079fe1c0581f67db162342c8
parent 3e34ad44e81722d0f2dd20192afdb4b954a370b7

Remove nullptr checks in FreeType deleters.

std::unique_ptr's dtor never call deleters with nullptr.

Change-Id: I308dcdb273527a582d1b579e4ec5c5eed33d8c94
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/70772
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>

core/fxge/fx_freetype.h

diff --git a/core/fxge/fx_freetype.h b/core/fxge/fx_freetype.h
index 2dd379a..a882421 100644
--- a/core/fxge/fx_freetype.h
+++ b/core/fxge/fx_freetype.h
@@ -24,17 +24,11 @@
 using FXFT_MM_VarPtr = FT_MM_Var*;
 
 struct FXFTFaceRecDeleter {
-  inline void operator()(FXFT_FaceRec* pRec) {
-    if (pRec)
-      FT_Done_Face(pRec);
-  }
+  inline void operator()(FXFT_FaceRec* pRec) { FT_Done_Face(pRec); }
 };
 
 struct FXFTLibraryRecDeleter {
-  inline void operator()(FXFT_LibraryRec* pRec) {
-    if (pRec)
-      FT_Done_FreeType(pRec);
-  }
+  inline void operator()(FXFT_LibraryRec* pRec) { FT_Done_FreeType(pRec); }
 };
 
 using ScopedFXFTFaceRec = std::unique_ptr<FXFT_FaceRec, FXFTFaceRecDeleter>;