COMPLETE REWRITE: Fix all syntax errors in pdf_comparator.py

- Completely rewrote load_pdf_pages function with cleaner structure
- Fixed indentation error at line 675 in barcode detection
- Fixed indentation error at line 1271 in debug function
- All syntax errors resolved - file now compiles successfully
- Simplified exception handling and improved code structure

pdf_comparator.py

@@ -371,47 +371,45 @@ def _contains_validation_text(text: str) -> bool:
     return "50 Carroll" in text
 
 def load_pdf_pages(path: str, dpi: int = 400, max_pages: int = 5) -> List[Image.Image]:
-    if _is_pdf(path):
-        # Try pdf2image with multiple poppler paths first
-        poppler_paths = ["/usr/bin", "/usr/local/bin", "/bin", None]
-        
-        for poppler_path in poppler_paths:
-            try:
-                if poppler_path:
-                    imgs = convert_from_path(path, dpi=dpi, first_page=1, last_page=max_pages, poppler_path=poppler_path)
-                else:
-                    imgs = convert_from_path(path, dpi=dpi, first_page=1, last_page=max_pages)
-                
-                if not imgs:
-                    continue
-                    
+    """Load PDF pages as images with fallback options"""
+    if not _is_pdf(path):
+        return [Image.open(path).convert("RGB")]
+    
+    # Try pdf2image first
+    poppler_paths = ["/usr/bin", "/usr/local/bin", "/bin", None]
+    
+    for poppler_path in poppler_paths:
+        try:
+            if poppler_path:
+                imgs = convert_from_path(path, dpi=dpi, first_page=1, last_page=max_pages, poppler_path=poppler_path)
+            else:
+                imgs = convert_from_path(path, dpi=dpi, first_page=1, last_page=max_pages)
+            
+            if imgs:
                 return [img.convert("RGB") for img in imgs]
+        except Exception:
+            if poppler_path is None:  # All pdf2image attempts failed
+                break
+            continue  # Try next path
+    
+    # Fallback to PyMuPDF
+    if HAS_PYMUPDF:
+        try:
+            doc = fitz.open(path)
+            pages = []
+            for page_num in range(min(len(doc), max_pages)):
+                page = doc[page_num]
+                mat = fitz.Matrix(dpi/72, dpi/72)
+                pix = page.get_pixmap(matrix=mat)
+                img_data = pix.tobytes("ppm")
+                img = Image.open(io.BytesIO(img_data))
+                pages.append(img.convert("RGB"))
+            doc.close()
+            return pages
         except Exception as e:
-                if poppler_path is None:  # All pdf2image attempts failed
-                    break
-                continue  # Try next path
-        
-        # Fallback to PyMuPDF if pdf2image fails
-        if HAS_PYMUPDF:
-            try:
-                doc = fitz.open(path)
-                pages = []
-                for page_num in range(min(len(doc), max_pages)):
-                    page = doc[page_num]
-                    mat = fitz.Matrix(dpi/72, dpi/72)  # Scale factor for DPI
-                    pix = page.get_pixmap(matrix=mat)
-                    img_data = pix.tobytes("ppm")
-                    img = Image.open(io.BytesIO(img_data))
-                    pages.append(img.convert("RGB"))
-                doc.close()
-                return pages
-        except Exception as e:
-                raise ValueError(f"Failed to convert PDF with both pdf2image and PyMuPDF. pdf2image error: poppler not found. PyMuPDF error: {str(e)}")
-        else:
-            raise ValueError(f"Failed to convert PDF to image with all poppler paths. Last error: poppler not found. PyMuPDF not available as fallback.")
-                
-        raise ValueError(f"No pages in PDF: {path}")
-    return [Image.open(path).convert("RGB")]
+            raise ValueError(f"Failed to convert PDF with both pdf2image and PyMuPDF. Error: {str(e)}")
+    
+    raise ValueError("Failed to convert PDF to image. No working method available.")
 
 def combine_pages_vertically(pages: List[Image.Image], spacing: int = 20) -> Image.Image:
     """Combine multiple pages into a single vertical image"""
@@ -667,7 +665,7 @@ def find_misspell_boxes_from_text(
                                 y1 = int(bbox[1] * scale_y) + (page_num * img_height)
                                 x2 = int(bbox[2] * scale_x)
                                 y2 = int(bbox[3] * scale_y) + (page_num * img_height)
-                else:
+                            else:
                                 # Use PDF coordinates directly (fallback)
                                 x1 = int(bbox[0])
                                 y1 = int(bbox[1]) + (page_num * 1000)
@@ -1270,7 +1268,7 @@ def debug_scan_pdf(pdf_path: str, outdir: str = "barcode_debug", max_pages=2):
                     rr = _decode_once(pil) or _decode_once(_binarize(pil))
                     if rr:
                         print(f"  Embedded image {ix+1}: {[(r.type, r.data) for r in rr]}")
-        except Exception as e:
+                except Exception as e:
                     print("  Embedded image error:", e)
     
     doc.close()