!466 [205_6] MuPDF：修复裁剪区域设置异常

Author: xjl12

devel/205_6.md

@@ -0,0 +1,14 @@
+# 205_6 MuPDF：修复裁剪区域设置异常
+## 如何测试
+启用MuPDF后编译、运行，菜单栏点击插入->图片->绘制图形，可以正常绘制各种图形
+
+## 2025/07/21
+### What
+问题根源在于，在绘制图形过程中，两个不同的`mupdf_renderer_rep`对象频繁、交替调用`set_clipping`函数，造成MuPDF内部图形状态管理异常，进而造成页面显示异常和崩溃。
+1. 重写`mupdf_renderer_rep::set_clipping()`函数，重点解决不同`mupdf_renderer_rep`对象交替调用时，MuPDF图形状态能够统一。
+2. 引入类静态变量`clip_active`，`clip_proc`统一管理裁剪设置的状态。
+    - `clip_active`指示裁剪区域是否已被设置，在`mupdf_renderer_rep::end`中，裁剪设置会还原。
+    - `clip_proc`保存上一次调用`set_clipping`函数的`mupdf_renderer_rep`对象中的MuPDF图形状态（`pdf_processor* proc`）地址，确保交替调用时，能够将上一次保存的图形状态顺利还原。
+
+### Why
+在启用MuPDF渲染器后，无法正常绘制图形，在绘制图形界面容易崩溃。

src/Plugins/MuPDF/mupdf_renderer.cpp

@@ -48,6 +48,9 @@ mupdf_document () {
   return doc;
 }
 
+bool           mupdf_renderer_rep::clip_active= false;
+pdf_processor* mupdf_renderer_rep::clip_proc;
+
 /******************************************************************************
  * Fitz pixmaps
  ******************************************************************************/
@@ -219,7 +222,6 @@ mupdf_renderer_rep::begin (void* handle) {
     current_width= -1.0;
     cfn          = "";
     in_text      = false;
-    clip_level   = 0;
 
     // outmost save of the graphics state
     proc->op_q (mupdf_context (), proc);
@@ -243,8 +245,11 @@ mupdf_renderer_rep::end () {
 
   if (proc) {
     // reset set_clipping calls in order to have well formed PDF.
-    while (clip_level--)
-      proc->op_Q (mupdf_context (), proc);
+    if (clip_active) {
+      clip_proc->op_Q (mupdf_context (), clip_proc);
+      clip_active= false;
+      clip_proc  = NULL;
+    }
     // outmost restore for the graphics state (see begin_page)
     proc->op_Q (mupdf_context (), proc);
 
@@ -328,25 +333,28 @@ mupdf_renderer_rep::set_clipping (SI x1, SI y1, SI x2, SI y2, bool restore) {
 
   outer_round (x1, y1, x2, y2);
   if (restore) {
-    // debug_convert << "restore clipping\n";
-    if (clip_level > 0) {
-      proc->op_Q (mupdf_context (), proc);
-      clip_level--;
-    }
     cfn= "";
   }
+  if (clip_proc != proc) {
+    if (clip_proc != NULL) clip_proc->op_Q (mupdf_context (), clip_proc);
+    clip_proc  = proc;
+    clip_active= false;
+  }
+  if (clip_active) {
+    proc->op_Q (mupdf_context (), proc);
+  }
   else {
-    // debug_convert << "set clipping\n";
-    proc->op_q (mupdf_context (), proc);
-    clip_level++;
-    float xx1= to_x (min (x1, x2));
-    float yy1= to_y (min (y1, y2));
-    float xx2= to_x (max (x1, x2));
-    float yy2= to_y (max (y1, y2));
-    proc->op_re (mupdf_context (), proc, xx1, yy1, xx2 - xx1, yy2 - yy1);
-    proc->op_W (mupdf_context (), proc);
-    proc->op_n (mupdf_context (), proc);
+    clip_active= true;
   }
+
+  proc->op_q (mupdf_context (), proc);
+  float xx1= to_x (min (x1, x2));
+  float yy1= to_y (min (y1, y2));
+  float xx2= to_x (max (x1, x2));
+  float yy2= to_y (max (y1, y2));
+  proc->op_re (mupdf_context (), proc, xx1, yy1, xx2 - xx1, yy2 - yy1);
+  proc->op_W (mupdf_context (), proc);
+  proc->op_n (mupdf_context (), proc);
 }
 
 /******************************************************************************

src/Plugins/MuPDF/mupdf_renderer.hpp

@@ -35,7 +35,6 @@ class mupdf_renderer_rep : public basic_renderer_rep {
   color  fg, bg;
   SI     lw;
   double current_width;
-  int    clip_level;
 
   //  pencil    pen;
   //  brush     bgb, fgb;
@@ -45,6 +44,9 @@ class mupdf_renderer_rep : public basic_renderer_rep {
   string cfn;
   float  fsize;
 
+  static bool           clip_active;
+  static pdf_processor* clip_proc;
+
   // geometry
 
   double to_x (SI x) {