[Bf-blender-cvs] [b38853e89f6] master: Cleanup: remove unused image code

Brecht Van Lommel noreply at git.blender.org
Fri May 17 18:00:23 CEST 2019


Commit: b38853e89f63fe65b57af003dc803611cda313e7
Author: Brecht Van Lommel
Date:   Fri May 17 15:46:53 2019 +0200
Branches: master
https://developer.blender.org/rBb38853e89f63fe65b57af003dc803611cda313e7

Cleanup: remove unused image code

===================================================================

M	source/blender/editors/space_image/image_buttons.c
M	source/blender/editors/space_image/image_ops.c

===================================================================

diff --git a/source/blender/editors/space_image/image_buttons.c b/source/blender/editors/space_image/image_buttons.c
index 752eedebe71..39cb5824627 100644
--- a/source/blender/editors/space_image/image_buttons.c
+++ b/source/blender/editors/space_image/image_buttons.c
@@ -650,23 +650,6 @@ static void image_multiview_cb(bContext *C, void *rnd_pt, void *UNUSED(arg_v))
   WM_event_add_notifier(C, NC_IMAGE | ND_DRAW, NULL);
 }
 
-#if 0
-static void image_freecache_cb(bContext *C, void *ima_v, void *unused)
-{
-  Scene *scene = CTX_data_scene(C);
-  BKE_image_free_anim_ibufs(ima_v, scene->r.cfra);
-  WM_event_add_notifier(C, NC_IMAGE, ima_v);
-}
-#endif
-
-#if 0
-static void image_user_change(bContext *C, void *iuser_v, void *unused)
-{
-  Scene *scene = CTX_data_scene(C);
-  BKE_image_user_calc_imanr(iuser_v, scene->r.cfra, 0);
-}
-#endif
-
 static void uiblock_layer_pass_buttons(
     uiLayout *layout, Image *image, RenderResult *rr, ImageUser *iuser, int w, short *render_slot)
 {
@@ -918,80 +901,6 @@ void uiTemplateImage(uiLayout *layout,
       uiItemL(layout, str, ICON_NONE);
 
       if (ima->type == IMA_TYPE_COMPOSITE) {
-        // XXX not working yet
-#if 0
-        iuser = ntree_get_active_iuser(scene->nodetree);
-        if (iuser) {
-          UI_block_align_begin(block);
-          uiDefIconTextBut(block,
-                           UI_BTYPE_BUT,
-                           B_SIMA_RECORD,
-                           ICON_REC,
-                           "Record",
-                           10,
-                           120,
-                           100,
-                           20,
-                           0,
-                           0,
-                           0,
-                           0,
-                           0,
-                           "");
-          uiDefIconTextBut(block,
-                           UI_BTYPE_BUT,
-                           B_SIMA_PLAY,
-                           ICON_PLAY,
-                           "Play",
-                           110,
-                           120,
-                           100,
-                           20,
-                           0,
-                           0,
-                           0,
-                           0,
-                           0,
-                           "");
-          but = uiDefBut(
-              block, UI_BTYPE_BUT, B_NOP, "Free Cache", 210, 120, 100, 20, 0, 0, 0, 0, 0, "");
-          UI_but_func_set(but, image_freecache_cb, ima, NULL);
-
-          if (iuser->frames)
-            BLI_snprintf(str, sizeof(str), "(%d) Frames:", iuser->framenr);
-          else
-            strcpy(str, "Frames:");
-          UI_block_align_begin(block);
-          uiDefButI(block,
-                    UI_BTYPE_NUM,
-                    imagechanged,
-                    str,
-                    10,
-                    90,
-                    150,
-                    20,
-                    &iuser->frames,
-                    0.0,
-                    MAXFRAMEF,
-                    0,
-                    0,
-                    "Number of images of a movie to use");
-          uiDefButI(block,
-                    UI_BTYPE_NUM,
-                    imagechanged,
-                    "StartFr:",
-                    160,
-                    90,
-                    150,
-                    20,
-                    &iuser->sfra,
-                    1.0,
-                    MAXFRAMEF,
-                    0,
-                    0,
-                    "Global starting frame of the movie");
-        }
-#endif
       }
       else if (ima->type == IMA_TYPE_R_RESULT) {
         /* browse layer/passes */
diff --git a/source/blender/editors/space_image/image_ops.c b/source/blender/editors/space_image/image_ops.c
index 533e39a27bc..eec9870ae45 100644
--- a/source/blender/editors/space_image/image_ops.c
+++ b/source/blender/editors/space_image/image_ops.c
@@ -181,13 +181,6 @@ static void sima_zoom_set_from_bounds(SpaceImage *sima, ARegion *ar, const rctf
   sima_zoom_set(sima, ar, size, NULL);
 }
 
-#if 0  // currently unused
-static bool image_poll(bContext *C)
-{
-  return (CTX_data_edit_image(C) != NULL);
-}
-#endif
-
 static bool space_image_buffer_exists_poll(bContext *C)
 {
   SpaceImage *sima = CTX_wm_space_image(C);
@@ -251,17 +244,6 @@ static bool space_image_file_exists_poll(bContext *C)
   return false;
 }
 
-#if 0 /* UNUSED */
-static bool space_image_poll(bContext *C)
-{
-  SpaceImage *sima = CTX_wm_space_image(C);
-  if (sima && sima->image) {
-    return true;
-  }
-  return false;
-}
-#endif
-
 bool space_image_main_region_poll(bContext *C)
 {
   SpaceImage *sima = CTX_wm_space_image(C);
@@ -3359,164 +3341,6 @@ void IMAGE_OT_curves_point_set(wmOperatorType *ot)
   RNA_def_property_flag(prop, PROP_SKIP_SAVE);
 }
 
-#if 0 /* Not ported to 2.5x yet */
-/******************** record composite operator *********************/
-
-typedef struct RecordCompositeData {
-  wmTimer *timer;
-  int old_cfra;
-  int sfra, efra;
-} RecordCompositeData;
-
-static int image_record_composite_apply(bContext *C, wmOperator *op)
-{
-  SpaceImage *sima = CTX_wm_space_image(C);
-  RecordCompositeData *rcd = op->customdata;
-  Scene *scene = CTX_data_scene(C);
-  ImBuf *ibuf;
-
-  WM_cursor_time(CTX_wm_window(C), scene->r.cfra);
-
-  // XXX scene->nodetree->test_break = BKE_blender_test_break;
-  // XXX scene->nodetree->test_break = NULL;
-
-  BKE_image_all_free_anim_ibufs(scene->r.cfra);
-  ntreeCompositExecTree(scene->nodetree,
-                        &scene->r,
-                        0,
-                        scene->r.cfra != rcd->old_cfra,
-                        &scene->view_settings,
-                        &scene->display_settings); /* 1 is no previews */
-
-  ED_area_tag_redraw(CTX_wm_area(C));
-
-  ibuf = BKE_image_acquire_ibuf(sima->image, &sima->iuser, NULL);
-  /* save memory in flipbooks */
-  if (ibuf)
-    imb_freerectfloatImBuf(ibuf);
-
-  BKE_image_release_ibuf(sima->image, ibuf, NULL);
-
-  scene->r.cfra++;
-
-  return (scene->r.cfra <= rcd->efra);
-}
-
-static int image_record_composite_init(bContext *C, wmOperator *op)
-{
-  SpaceImage *sima = CTX_wm_space_image(C);
-  Scene *scene = CTX_data_scene(C);
-  RecordCompositeData *rcd;
-
-  if (sima->iuser.frames < 2)
-    return 0;
-  if (scene->nodetree == NULL)
-    return 0;
-
-  op->customdata = rcd = MEM_callocN(sizeof(RecordCompositeData), "ImageRecordCompositeData");
-
-  rcd->old_cfra = scene->r.cfra;
-  rcd->sfra = sima->iuser.sfra;
-  rcd->efra = sima->iuser.sfra + sima->iuser.frames - 1;
-  scene->r.cfra = rcd->sfra;
-
-  return 1;
-}
-
-static void image_record_composite_exit(bContext *C, wmOperator *op)
-{
-  Scene *scene = CTX_data_scene(C);
-  SpaceImage *sima = CTX_wm_space_image(C);
-  RecordCompositeData *rcd = op->customdata;
-
-  scene->r.cfra = rcd->old_cfra;
-
-  WM_cursor_modal_restore(CTX_wm_window(C));
-
-  if (rcd->timer)
-    WM_event_remove_timer(CTX_wm_manager(C), CTX_wm_window(C), rcd->timer);
-
-  WM_event_add_notifier(C, NC_IMAGE | NA_EDITED, sima->image);
-
-  // XXX play_anim(0);
-  // XXX allqueue(REDRAWNODE, 1);
-
-  MEM_freeN(rcd);
-}
-
-static int image_record_composite_exec(bContext *C, wmOperator *op)
-{
-  if (!image_record_composite_init(C, op))
-    return OPERATOR_CANCELLED;
-
-  while (image_record_composite_apply(C, op)) {
-  }
-
-  image_record_composite_exit(C, op);
-
-  return OPERATOR_FINISHED;
-}
-
-static int image_record_composite_invoke(bContext *C, wmOperator *op, const wmEvent *UNUSED(event))
-{
-  RecordCompositeData *rcd;
-
-  if (!image_record_composite_init(C, op))
-    return OPERATOR_CANCELLED;
-
-  rcd = op->customdata;
-  rcd->timer = WM_event_add_timer(CTX_wm_manager(C), CTX_wm_window(C), TIMER, 0.0f);
-  WM_event_add_modal_handler(C, op);
-
-  if (!image_record_composite_apply(C, op))
-    return OPERATOR_FINISHED;
-
-  return OPERATOR_RUNNING_MODAL;
-}
-
-static int image_record_composite_modal(bContext *C, wmOperator *op, const wmEvent *event)
-{
-  RecordCompositeData *rcd = op->customdata;
-
-  switch (event->type) {
-    case TIMER:
-      if (rcd->timer == event->customdata) {
-        if (!image_record_composite_apply(C, op)) {
-          image_record_composite_exit(C, op);
-          return OPERATOR_FINISHED;
-        }
-      }
-      break;
-    case ESCKEY:
-      image_record_composite_exit(C, op);
-      return OPERATOR_FINISHED;
-  }
-
-  return OPERATOR_RUNNING_MODAL;
-}
-
-static void image_record_composite_cancel(bContext *C, wmOperator *op)
-{
-  image_record_composite_exit(C, op);
-  return OPERATOR_CANCELLED;
-}
-
-void IMAGE_OT_record_composite(wmOperatorType *ot)
-{
-  /* identifiers */
-  ot->name = "Record Composite";
-  ot->idname = "IMAGE_OT_record_composite";
-
-  /* api callbacks */
-  ot->exec = image_record_composite_exec;
-  ot->invoke = image_record_composite_invoke;
-  ot->modal = image_record_composite_modal;
-  ot->cancel = image_record_composite_cancel;
-  ot->poll = space_image_buffer_exists_poll;
-}
-
-#endif
-
 /********************* cycle render slot operator *********************/
 
 static bool image_cycle_render_slot_poll(bContext *C)



More information about the Bf-blender-cvs mailing list