Fixes for alpha mode do_versions code
authorSergey Sharybin <sergey.vfx@gmail.com>
Tue, 19 Feb 2013 08:37:08 +0000 (08:37 +0000)
committerSergey Sharybin <sergey.vfx@gmail.com>
Tue, 19 Feb 2013 08:37:08 +0000 (08:37 +0000)
Before this change only old flag "Premultiply" was used to
detect alpha mode, which is not enough actually.

Now the logic here is:

- If "Premultiply" was enabled it is likely float image with
  straight alpha, which shall be premultiplied before usage.

  In this case image/sequence Alpha Mode is set to Straight.

- Otherwise use default alpha mode for image format based on
  an extension. This could fail in some cases like TIFF, but
  this wasn't handled fully correct in older blender anyway.

Initial discovered issue was that EXR images saved in older
Blender versions were set to Straight alpha mode, which is
obviously a straight way to lots of headache.

source/blender/blenkernel/BKE_image.h
source/blender/blenkernel/BKE_sequencer.h
source/blender/blenkernel/intern/image.c
source/blender/blenkernel/intern/sequencer.c
source/blender/blenloader/intern/readfile.c

index d12b048..dd7e062 100644 (file)
@@ -152,6 +152,9 @@ void BKE_image_pool_free(struct ImagePool *pool);
 struct ImBuf *BKE_image_pool_acquire_ibuf(struct Image *ima, struct ImageUser *iuser, struct ImagePool *pool);
 void BKE_image_pool_release_ibuf(struct Image *ima, struct ImBuf *ibuf, struct ImagePool *pool);
 
+/* set an alpha mode based on file extension */
+void BKE_image_alpha_mode_from_extension(struct Image *image);
+
 /* returns a new image or NULL if it can't load */
 struct Image *BKE_image_load(struct Main *bmain, const char *filepath);
 /* returns existing Image when filename/type is same (frame optional) */
index e3d9c51..eef134a 100644 (file)
@@ -356,6 +356,8 @@ typedef struct SeqLoadInfo {
 typedef struct Sequence *(*SeqLoadFunc)(struct bContext *, ListBase *, struct SeqLoadInfo *);
 
 struct Sequence *BKE_sequence_alloc(ListBase *lb, int cfra, int machine);
+
+void BKE_sequence_alpha_mode_from_extension(struct Sequence *seq);
 void BKE_sequence_init_colorspace(struct Sequence *seq);
 
 struct Sequence *BKE_sequencer_add_image_strip(struct bContext *C, ListBase *seqbasep, struct SeqLoadInfo *seq_load);
index de85cb5..3655afd 100644 (file)
@@ -571,6 +571,20 @@ static void image_init_color_management(Image *ima)
        }
 }
 
+void BKE_image_alpha_mode_from_extension(Image *image)
+{
+       if (BLI_testextensie(image->name, ".exr") ||
+           BLI_testextensie(image->name, ".cin") ||
+           BLI_testextensie(image->name, ".dpx") ||
+           BLI_testextensie(image->name, ".hdr"))
+       {
+               image->alpha_mode = IMA_ALPHA_PREMUL;
+       }
+       else {
+               image->alpha_mode = IMA_ALPHA_STRAIGHT;
+       }
+}
+
 Image *BKE_image_load(Main *bmain, const char *filepath)
 {
        Image *ima;
index 4539372..6861828 100644 (file)
@@ -3955,6 +3955,24 @@ Sequence *BKE_sequence_alloc(ListBase *lb, int cfra, int machine)
        return seq;
 }
 
+void BKE_sequence_alpha_mode_from_extension(Sequence *seq)
+{
+       if (seq->strip && seq->strip->stripdata) {
+               char *name = seq->strip->stripdata->name;
+
+               if (BLI_testextensie(name, ".exr") ||
+                   BLI_testextensie(name, ".cin") ||
+                   BLI_testextensie(name, ".dpx") ||
+                   BLI_testextensie(name, ".hdr"))
+                       {
+                               seq->alpha_mode = IMA_ALPHA_PREMUL;
+                       }
+               else {
+                       seq->alpha_mode = IMA_ALPHA_STRAIGHT;
+               }
+       }
+}
+
 void BKE_sequence_init_colorspace(Sequence *seq)
 {
        if (seq->strip && seq->strip->stripdata) {
index 94dfb0c..4471e45 100644 (file)
@@ -8653,8 +8653,12 @@ static void do_versions(FileData *fd, Library *lib, Main *main)
 
                        SEQ_BEGIN (scene->ed, seq)
                        {
-                               if (seq->flag & SEQ_MAKE_PREMUL)
+                               if (seq->flag & SEQ_MAKE_PREMUL) {
                                        seq->alpha_mode = SEQ_ALPHA_STRAIGHT;
+                               }
+                               else {
+                                       BKE_sequence_alpha_mode_from_extension(seq);
+                               }
                        }
                        SEQ_END
 
@@ -8680,8 +8684,12 @@ static void do_versions(FileData *fd, Library *lib, Main *main)
                }
 
                for (image = main->image.first; image; image = image->id.next) {
-                       if (image->flag & IMA_DO_PREMUL)
+                       if (image->flag & IMA_DO_PREMUL) {
                                image->alpha_mode = IMA_ALPHA_STRAIGHT;
+                       }
+                       else {
+                               BKE_image_alpha_mode_from_extension(image);
+                       }
 
                        image->flag &= ~IMA_DONE_TAG;
                }