Extern in .cpp files is a code smell.

fpdfsdk/src/fpdftext.cpp

 // Copyright 2014 PDFium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
 
 #include "../../core/include/fpdfdoc/fpdf_doc.h"
 #include "../../core/include/fpdftext/fpdf_text.h"
 #include "../../public/fpdf_text.h"
 
 #ifdef _WIN32
 #include <tchar.h>
 #endif
 
-// jabdelmalek: commented out to build on Linux.  Not used.
-// extern HANDLE g_hModule;
-
 DLLEXPORT FPDF_TEXTPAGE STDCALL FPDFText_LoadPage(FPDF_PAGE page) {
   if (!page)
     return NULL;
   IPDF_TextPage* textpage = NULL;
   CPDF_ViewerPreferences viewRef(((CPDF_Page*)page)->m_pDocument);
   textpage =
       IPDF_TextPage::CreateTextPage((CPDF_Page*)page, viewRef.IsDirectionR2L());
   textpage->ParseTextPage();
   return textpage;
 }
@@ skipping 250 matching lines @@
     CFX_FloatRect rect = rectArray.GetAt(rect_index);
     *left = rect.left;
     *right = rect.right;
     *top = rect.top;
     *bottom = rect.bottom;
   }
 }
 DLLEXPORT void STDCALL FPDFLink_CloseWebLinks(FPDF_PAGELINK link_page) {
   delete (IPDF_LinkExtract*)link_page;
 }