fix(loader): split builtin wrapper TU to avoid pcc ICE

Author: saitoha

src/loader-builtin.c

@@ -5714,21 +5714,22 @@ sixel_builtin_load_stbi_path(
 }
 
 SIXELSTATUS
-load_with_builtin(
-    sixel_chunk_t const *pchunk,
-    int fstatic,
-    int fuse_palette,
-    int reqcolors,
-    unsigned char *bgcolor,
-    int bgcolor_source,
-    int loop_control,
-    int start_frame_no_set,
-    int start_frame_no_override,
-    int enable_cms,
-    int bmp_info40_mode,
-    sixel_load_image_function fn_load,
-    void *context)
+sixel_builtin_load_with_builtin_impl(
+    sixel_builtin_load_with_builtin_args_t const *args)
 {
+    sixel_chunk_t const *pchunk;
+    int fstatic;
+    int fuse_palette;
+    int reqcolors;
+    unsigned char *bgcolor;
+    int bgcolor_source;
+    int loop_control;
+    int start_frame_no_set;
+    int start_frame_no_override;
+    int enable_cms;
+    int bmp_info40_mode;
+    sixel_load_image_function fn_load;
+    void *context;
     SIXELSTATUS status;
     unsigned char *pixels;
     sixel_frame_t *frame;
@@ -5752,11 +5753,27 @@ load_with_builtin(
     int apply_start_frame;
     int pnm_pixelformat;
 
+    if (args == NULL) {
+        return SIXEL_BAD_ARGUMENT;
+    }
+    pchunk = args->pchunk;
+    fstatic = args->fstatic;
+    fuse_palette = args->fuse_palette;
+    reqcolors = args->reqcolors;
+    bgcolor = args->bgcolor;
+    bgcolor_source = args->bgcolor_source;
+    loop_control = args->loop_control;
+    start_frame_no_set = args->start_frame_no_set;
+    start_frame_no_override = args->start_frame_no_override;
+    enable_cms = args->enable_cms;
+    bmp_info40_mode = args->bmp_info40_mode;
+    fn_load = args->fn_load;
+    context = args->context;
     status = SIXEL_BAD_INPUT;
     pixels = NULL;
     frame = NULL;
-    stb_context = (stbi__context){ 0 };
-    ri = (stbi__result_info){ 0 };
+    memset(&stb_context, 0, sizeof(stb_context));
+    memset(&ri, 0, sizeof(ri));
     chunk_size = 0;
     is_png = 0;
     is_jpeg = 0;

src/loader-builtin.h

@@ -33,11 +33,31 @@
 #include "chunk.h"
 #include "loader-component.h"
 
+typedef struct sixel_builtin_load_with_builtin_args {
+    sixel_chunk_t const       *pchunk;
+    int                        fstatic;
+    int                        fuse_palette;
+    int                        reqcolors;
+    unsigned char             *bgcolor;
+    int                        bgcolor_source;
+    int                        loop_control;
+    int                        start_frame_no_set;
+    int                        start_frame_no_override;
+    int                        enable_cms;
+    int                        bmp_info40_mode;
+    sixel_load_image_function  fn_load;
+    void                      *context;
+} sixel_builtin_load_with_builtin_args_t;
+
 SIXELSTATUS
 sixel_loader_builtin_new(
     sixel_allocator_t *allocator,
     sixel_loader_component_t **ppcomponent);
 
+SIXELSTATUS
+sixel_builtin_load_with_builtin_impl(
+    sixel_builtin_load_with_builtin_args_t const *args);
+
 SIXELSTATUS load_with_builtin(
     sixel_chunk_t const       *pchunk,
     int                        fstatic,

src/loader-component.c

@@ -27,6 +27,7 @@
 #endif
 
 #include "loader-component.h"
+#include "loader-builtin.h"
 
 void
 sixel_loader_component_ref(sixel_loader_component_t *component)
@@ -88,6 +89,40 @@ sixel_loader_component_get_name(sixel_loader_component_t const *component)
     return component->vtbl->name(component);
 }
 
+SIXELSTATUS
+load_with_builtin(
+    sixel_chunk_t const *pchunk,
+    int fstatic,
+    int fuse_palette,
+    int reqcolors,
+    unsigned char *bgcolor,
+    int bgcolor_source,
+    int loop_control,
+    int start_frame_no_set,
+    int start_frame_no_override,
+    int enable_cms,
+    int bmp_info40_mode,
+    sixel_load_image_function fn_load,
+    void *context)
+{
+    sixel_builtin_load_with_builtin_args_t args;
+
+    args.pchunk = pchunk;
+    args.fstatic = fstatic;
+    args.fuse_palette = fuse_palette;
+    args.reqcolors = reqcolors;
+    args.bgcolor = bgcolor;
+    args.bgcolor_source = bgcolor_source;
+    args.loop_control = loop_control;
+    args.start_frame_no_set = start_frame_no_set;
+    args.start_frame_no_override = start_frame_no_override;
+    args.enable_cms = enable_cms;
+    args.bmp_info40_mode = bmp_info40_mode;
+    args.fn_load = fn_load;
+    args.context = context;
+    return sixel_builtin_load_with_builtin_impl(&args);
+}
+
 /* emacs Local Variables:      */
 /* emacs mode: c               */
 /* emacs tab-width: 4          */