Cleanup: take Main argument for copy
authorCampbell Barton <ideasman42@gmail.com>
Thu, 3 Mar 2016 02:35:21 +0000 (13:35 +1100)
committerCampbell Barton <ideasman42@gmail.com>
Thu, 3 Mar 2016 02:35:21 +0000 (13:35 +1100)
source/blender/blenkernel/BKE_blender.h
source/blender/blenkernel/intern/blender.c
source/blender/blenloader/BLO_writefile.h
source/blender/blenloader/intern/writefile.c
source/blender/editors/space_view3d/view3d_ops.c

index 4f8184ba66c12d443d8721c3d08ecfa6b288a9ed..0a35d0aba57ea160ce5e02fc9b71f320f2c3a211 100644 (file)
@@ -104,10 +104,10 @@ extern bool          BKE_undo_save_file(const char *filename);
 extern struct Main  *BKE_undo_get_main(struct Scene **r_scene);
 
 /* copybuffer */
-void BKE_copybuffer_begin(struct Main *bmain);
+void BKE_copybuffer_begin(struct Main *bmain_src);
 void BKE_copybuffer_tag_ID(struct ID *id);
-int BKE_copybuffer_save(const char *filename, struct ReportList *reports);
-int BKE_copybuffer_paste(struct bContext *C, const char *libname, const short flag, struct ReportList *reports);
+bool BKE_copybuffer_save(struct Main *bmain_src, const char *filename, struct ReportList *reports);
+bool BKE_copybuffer_paste(struct bContext *C, const char *libname, const short flag, struct ReportList *reports);
 
 #ifdef __cplusplus
 }
index b9cf21a7a3a976a64c4138703fbc7183e4244f24..e1db35f50cbe49abfcf7eb3946b6357cf74eb7ef 100644 (file)
@@ -639,7 +639,10 @@ int blender_test_break(void)
 }
 
 
-/* ***************** GLOBAL UNDO *************** */
+/* -------------------------------------------------------------------- */
+
+/** \name Global Undo
+ * \{ */
 
 #define UNDO_DISK   0
 
@@ -957,14 +960,18 @@ Main *BKE_undo_get_main(Scene **r_scene)
        return mainp;
 }
 
-/* ************** copy paste .blend, partial saves ********** */
+/** \} */
+
+
+/* -------------------------------------------------------------------- */
 
-/* assumes data is in G.main */
+/** \name Copy/Paste `.blend`, partial saves.
+ * \{ */
 
-void BKE_copybuffer_begin(Main *bmain)
+void BKE_copybuffer_begin(Main *bmain_src)
 {
        /* set all id flags to zero; */
-       BKE_main_id_tag_all(bmain, LIB_TAG_NEED_EXPAND | LIB_TAG_DOIT, false);
+       BKE_main_id_tag_all(bmain_src, LIB_TAG_NEED_EXPAND | LIB_TAG_DOIT, false);
 }
 
 void BKE_copybuffer_tag_ID(ID *id)
@@ -972,7 +979,7 @@ void BKE_copybuffer_tag_ID(ID *id)
        id->tag |= LIB_TAG_NEED_EXPAND | LIB_TAG_DOIT;
 }
 
-static void copybuffer_doit(void *UNUSED(handle), Main *UNUSED(bmain), void *vid)
+static void copybuffer_doit(void *UNUSED(handle), Main *UNUSED(bmain_src), void *vid)
 {
        if (vid) {
                ID *id = vid;
@@ -982,70 +989,75 @@ static void copybuffer_doit(void *UNUSED(handle), Main *UNUSED(bmain), void *vid
        }
 }
 
-/* frees main in end */
-int BKE_copybuffer_save(const char *filename, ReportList *reports)
+/**
+ * \return Success.
+ */
+bool BKE_copybuffer_save(Main *bmain_src, const char *filename, ReportList *reports)
 {
-       Main *mainb = MEM_callocN(sizeof(Main), "copybuffer");
-       ListBase *lbarray[MAX_LIBARRAY], *fromarray[MAX_LIBARRAY];
+       /* frees main in end */
+       Main *bmain_dst = MEM_callocN(sizeof(Main), "copybuffer");
+       ListBase *lbarray_dst[MAX_LIBARRAY], *lbarray_src[MAX_LIBARRAY];
        int a, retval;
        
        /* path backup/restore */
        void     *path_list_backup;
        const int path_list_flag = (BKE_BPATH_TRAVERSE_SKIP_LIBRARY | BKE_BPATH_TRAVERSE_SKIP_MULTIFILE);
 
-       path_list_backup = BKE_bpath_list_backup(G.main, path_list_flag);
+       path_list_backup = BKE_bpath_list_backup(bmain_src, path_list_flag);
 
        BLO_main_expander(copybuffer_doit);
-       BLO_expand_main(NULL, G.main);
+       BLO_expand_main(NULL, bmain_src);
        
        /* move over all tagged blocks */
-       set_listbasepointers(G.main, fromarray);
-       a = set_listbasepointers(mainb, lbarray);
+       set_listbasepointers(bmain_src, lbarray_src);
+       a = set_listbasepointers(bmain_dst, lbarray_dst);
        while (a--) {
                ID *id, *nextid;
-               ListBase *lb1 = lbarray[a], *lb2 = fromarray[a];
+               ListBase *lb_dst = lbarray_dst[a], *lb_src = lbarray_src[a];
                
-               for (id = lb2->first; id; id = nextid) {
+               for (id = lb_src->first; id; id = nextid) {
                        nextid = id->next;
                        if (id->tag & LIB_TAG_DOIT) {
-                               BLI_remlink(lb2, id);
-                               BLI_addtail(lb1, id);
+                               BLI_remlink(lb_src, id);
+                               BLI_addtail(lb_dst, id);
                        }
                }
        }
        
        
        /* save the buffer */
-       retval = BLO_write_file(mainb, filename, G_FILE_RELATIVE_REMAP, reports, NULL);
+       retval = BLO_write_file(bmain_dst, filename, G_FILE_RELATIVE_REMAP, reports, NULL);
        
        /* move back the main, now sorted again */
-       set_listbasepointers(G.main, lbarray);
-       a = set_listbasepointers(mainb, fromarray);
+       set_listbasepointers(bmain_src, lbarray_dst);
+       a = set_listbasepointers(bmain_dst, lbarray_src);
        while (a--) {
                ID *id;
-               ListBase *lb1 = lbarray[a], *lb2 = fromarray[a];
+               ListBase *lb_dst = lbarray_dst[a], *lb_src = lbarray_src[a];
                
-               while ((id = BLI_pophead(lb2))) {
-                       BLI_addtail(lb1, id);
-                       id_sort_by_name(lb1, id);
+               while ((id = BLI_pophead(lb_src))) {
+                       BLI_addtail(lb_dst, id);
+                       id_sort_by_name(lb_dst, id);
                }
        }
        
-       MEM_freeN(mainb);
+       MEM_freeN(bmain_dst);
        
        /* set id flag to zero; */
-       BKE_main_id_tag_all(G.main, LIB_TAG_NEED_EXPAND | LIB_TAG_DOIT, false);
+       BKE_main_id_tag_all(bmain_src, LIB_TAG_NEED_EXPAND | LIB_TAG_DOIT, false);
        
        if (path_list_backup) {
-               BKE_bpath_list_restore(G.main, path_list_flag, path_list_backup);
+               BKE_bpath_list_restore(bmain_src, path_list_flag, path_list_backup);
                BKE_bpath_list_free(path_list_backup);
        }
 
        return retval;
 }
 
-/* return success (1) */
-int BKE_copybuffer_paste(bContext *C, const char *libname, const short flag, ReportList *reports)
+/**
+ * \return Success.
+ */
+bool BKE_copybuffer_paste(bContext *C, const char *libname, const short flag, ReportList *reports)
 {
        Main *bmain = CTX_data_main(C);
        Scene *scene = CTX_data_scene(C);
@@ -1058,7 +1070,7 @@ int BKE_copybuffer_paste(bContext *C, const char *libname, const short flag, Rep
        
        if (bh == NULL) {
                /* error reports will have been made by BLO_blendhandle_from_file() */
-               return 0;
+               return false;
        }
 
        BKE_scene_base_deselect_all(scene);
@@ -1094,5 +1106,7 @@ int BKE_copybuffer_paste(bContext *C, const char *libname, const short flag, Rep
        BLO_blendhandle_close(bh);
        /* remove library... */
        
-       return 1;
+       return true;
 }
+
+/** \} */
index 0d66eb743aa655737ccb5cf52e028a7371422d78..af3bc2dbdcd85e622c796b908839317d33abcee0 100644 (file)
@@ -38,9 +38,11 @@ struct MemFile;
 struct Main;
 struct ReportList;
 
-extern int BLO_write_file(struct Main *mainvar, const char *filepath, int write_flags,
+extern bool BLO_write_file(
+        struct Main *mainvar, const char *filepath, int write_flags,
         struct ReportList *reports, const struct BlendThumbnail *thumb);
-extern int BLO_write_file_mem(struct Main *mainvar, struct MemFile *compare, struct MemFile *current, int write_flags);
+extern bool BLO_write_file_mem(
+        struct Main *mainvar, struct MemFile *compare, struct MemFile *current, int write_flags);
 
 #endif
 
index e8db2d3679bc223ccffbf5e28a225c608c4efa3c..4871dc2f161b164a0c36d3e51ed476d265a9822d 100644 (file)
@@ -3825,9 +3825,12 @@ static bool do_history(const char *name, ReportList *reports)
        return 0;
 }
 
-/* return: success (1) */
-int BLO_write_file(
-        Main *mainvar, const char *filepath, int write_flags, ReportList *reports, const BlendThumbnail *thumb)
+/**
+ * \return Success.
+ */
+bool BLO_write_file(
+        Main *mainvar, const char *filepath, int write_flags,
+        ReportList *reports, const BlendThumbnail *thumb)
 {
        char tempname[FILE_MAX+1];
        int err, write_user_block;
@@ -3925,14 +3928,14 @@ int BLO_write_file(
        return 1;
 }
 
-/* return: success (1) */
-int BLO_write_file_mem(Main *mainvar, MemFile *compare, MemFile *current, int write_flags)
+/**
+ * \return Success.
+ */
+bool BLO_write_file_mem(Main *mainvar, MemFile *compare, MemFile *current, int write_flags)
 {
        int err;
 
        err = write_file_handle(mainvar, NULL, compare, current, 0, write_flags, NULL);
-       
-       if (err==0) return 1;
-       return 0;
-}
 
+       return (err == 0);
+}
index c31d9c197e7add239317e2eb84b9693ee2663a01..a5411da131b1adc3adc97dae611e992ba53d89ef 100644 (file)
@@ -92,7 +92,7 @@ static int view3d_copybuffer_exec(bContext *C, wmOperator *op)
        }
        
        BLI_make_file_string("/", str, BKE_tempdir_base(), "copybuffer.blend");
-       BKE_copybuffer_save(str, op->reports);
+       BKE_copybuffer_save(bmain, str, op->reports);
        
        BKE_report(op->reports, RPT_INFO, "Copied selected objects to buffer");