Engine: fix a case when new sprite is matching object graphic property

Another problem with dynamic sprite since 3.6.1...
Case:
- an object has a dynamic sprite assigned;
- dynamic sprite gets deleted, object redrawn, its texture updated etc, all good (displays safe placeholder);
- new dynamic sprite created, with same index by coincidence, and assigned to the object;
- object fails to redraw, because sprite's number is the same as saved in property value, and there's no notification on sprite *creation*.

Solution:
- make a "sprite updated" notification on sprite creation too.

Author: ivan-mogilko

Engine/ac/dynamicsprite.cpp

@@ -113,7 +113,6 @@ void DynamicSprite_Resize(ScriptDynamicSprite *sds, int width, int height) {
         RectWH(0, 0, width, height));
 
     add_dynamic_sprite(sds->slot, std::move(new_pic));
-    game_sprite_updated(sds->slot);
 }
 
 void DynamicSprite_Flip(ScriptDynamicSprite *sds, int direction) {
@@ -130,7 +129,6 @@ void DynamicSprite_Flip(ScriptDynamicSprite *sds, int direction) {
     new_pic->FlipBlt(sprite, 0, 0, flip);
 
     add_dynamic_sprite(sds->slot, std::move(new_pic));
-    game_sprite_updated(sds->slot);
 }
 
 void DynamicSprite_CopyTransparencyMask(ScriptDynamicSprite *sds, int sourceSprite) {
@@ -153,7 +151,6 @@ void DynamicSprite_CopyTransparencyMask(ScriptDynamicSprite *sds, int sourceSpri
 
     // set the target's alpha channel depending on the source
     BitmapHelper::CopyTransparency(target, source, target->GetColorDepth() == 32, source->GetColorDepth() == 32);
-    game_sprite_updated(sds->slot);
 }
 
 void DynamicSprite_ChangeCanvasSize(ScriptDynamicSprite *sds, int width, int height, int x, int y) 
@@ -169,7 +166,6 @@ void DynamicSprite_ChangeCanvasSize(ScriptDynamicSprite *sds, int width, int hei
     new_pic->Blit(sprite, 0, 0, x, y, sprite->GetWidth(), sprite->GetHeight());
 
     add_dynamic_sprite(sds->slot, std::move(new_pic));
-    game_sprite_updated(sds->slot);
 }
 
 void DynamicSprite_Crop(ScriptDynamicSprite *sds, int x1, int y1, int width, int height) {
@@ -187,7 +183,6 @@ void DynamicSprite_Crop(ScriptDynamicSprite *sds, int x1, int y1, int width, int
 
     // replace the bitmap in the sprite set
     add_dynamic_sprite(sds->slot, std::move(new_pic));
-    game_sprite_updated(sds->slot);
 }
 
 void DynamicSprite_Rotate(ScriptDynamicSprite *sds, int angle, int width, int height) {
@@ -215,7 +210,6 @@ void DynamicSprite_Rotate(ScriptDynamicSprite *sds, int angle, int width, int he
 
     // replace the bitmap in the sprite set
     add_dynamic_sprite(sds->slot, std::move(new_pic));
-    game_sprite_updated(sds->slot);
 }
 
 void DynamicSprite_Tint(ScriptDynamicSprite *sds, int red, int green, int blue, int saturation, int luminance) 
@@ -227,7 +221,6 @@ void DynamicSprite_Tint(ScriptDynamicSprite *sds, int red, int green, int blue,
     tint_image(new_pic.get(), source, red, green, blue, saturation, GfxDef::Value100ToValue250(luminance));
 
     add_dynamic_sprite(sds->slot, std::move(new_pic));
-    game_sprite_updated(sds->slot);
 }
 
 int DynamicSprite_SaveToFile(ScriptDynamicSprite *sds, const char* namm)
@@ -457,6 +450,11 @@ int add_dynamic_sprite(int slot, std::unique_ptr<Bitmap> image, uint32_t extra_f
     uint32_t flags = SPF_DYNAMICALLOC | extra_flags;
     if (!spriteset.SetSprite(slot, std::move(image), flags))
         return 0; // failed to add the sprite, bad image or realloc failed
+
+    // Notify a new (normal) dynamic sprite in case some objects
+    // have this number assigned to their Graphic property
+    if ((extra_flags & SPF_OBJECTOWNED) == 0)
+        game_sprite_updated(slot);
     return slot;
 }
 

Engine/ac/game.cpp

@@ -1595,6 +1595,8 @@ void game_sprite_updated(int sprnum, bool deleted)
 
     // GUI still have a special draw route, so cannot rely on object caches;
     // will have to do a per-GUI and per-control check.
+    // TODO: devise a way to speed this iteration up, perhaps link GUI objects
+    // to the sprite notification block somehow, etc...?
     //
     // gui backgrounds
     for (auto &gui : guis)