<?xml version="1.0"?>
<rss version="2.0">
	<channel>
		<title>Drawing onto bitmap crashes when resizing the display</title>
		<link>http://www.allegro.cc/forums/view/616905</link>
		<description>Allegro.cc Forum Thread</description>
		<webMaster>matthew@allegro.cc (Matthew Leverton)</webMaster>
		<lastBuildDate>Thu, 25 May 2017 11:35:33 +0000</lastBuildDate>
	</channel>
	<item>
		<description><![CDATA[<div class="mockup v2"><p>Hi all,</p><p>When I resize the display I acknowledge it with al_acknowledge_resize but if I try to draw to a bitmap I created at the start of the program again it crashes.  If I try to recreate this bitmap it still crashes.  I have already narrowed it down to this; when the al_set_target_bitmap( bitmap ) is removed it doesn&#39;t crash.</p><p>Was this a known bug and has it been fixed?  </p><p>I&#39;m on Allegro 5.2.0.  Windows 10, DirectX driver (I assume)</p><p>It&#39;s been a while since I attempted to build Allegro or anything written in C++ for that matter (I work in SwiftForth).  I&#39;d prefer not to go down that rabbit hole again... (though I might finally get on the contributor train, since I&#39;m pretty good at git now...)</p><p>I found a 5.2.1 DLL Siegelord posted but it fails to load; I suspect due to missing dependencies (which is odd because it was the monolith build...)  The thread was about a bug with the builtin font after resizing so it could be related.  (<a href="https://www.allegro.cc/forums/thread/616340">https://www.allegro.cc/forums/thread/616340</a>)</p><p>UPDATE:</p><p>I found the 5.2.2 release and fixed the loading problem.  Looks like the bitmap bug was fixed.  </p><p>After confirming that, I thought of an alternative approach; create the window at the native resolution and DON&#39;T acknowledge resizes.  Immediately resize it to the desired size.  There&#39;ll be a blip where you see a big window but this is minor.  This way contents of bitmaps are not lost and there will be no delay when resizing or fullscreening.
</p></div>]]>
		</description>
		<author>no-reply@allegro.cc (roger levy)</author>
		<pubDate>Wed, 24 May 2017 23:33:13 +0000</pubDate>
	</item>
	<item>
		<description><![CDATA[<div class="mockup v2"><p>In my own game, when I change the display, I call a function to shut down the current display; <span class="source-code">shutdown_display<span class="k2">(</span><span class="k1">true</span><span class="k2">)</span></span>, which involves destroying all bitmaps that were loaded (as if the game was shutting down), I then call an <span class="source-code">init_display<span class="k2">(</span><span class="k2">)</span></span> function which is called when the game starts, it has a flag to reset the display which is passed to it, and it then loads all the graphics back in.</p><p>It works just fine that way without problems. (see my Deluxe Pacman 2 game in the link below, it is free, if you run it and change the display you will see it works perfectly).</p><p>I&#39;ve never tried to do it differently.  If it works, I don&#39;t mess with it. <img src="http://www.allegro.cc/forums/smileys/wink.gif" alt=";)" /></p><p>My functions I use below... I call <span class="source-code">shutdown_display<span class="k2">(</span><span class="k2">)</span></span> then <span class="source-code">init_display<span class="k2">(</span><span class="k1">true</span><span class="k2">)</span></span> when it is resized...</p><div class="source-code snippet"><div class="inner"><pre><span class="k1">void</span> shutdown_display<span class="k2">(</span><span class="k1">void</span><span class="k2">)</span>
<span class="k2">{</span>
   <span class="c">/* Destroy loaded bitmaps here */</span>

   <a href="http://www.allegro.cc/manual/al_unregister_event_source"><span class="a">al_unregister_event_source</span></a><span class="k2">(</span>event_queue, <a href="http://www.allegro.cc/manual/al_get_display_event_source"><span class="a">al_get_display_event_source</span></a><span class="k2">(</span>setting.screen<span class="k2">)</span><span class="k2">)</span><span class="k2">;</span>

   <span class="k1">if</span><span class="k2">(</span>setting.screen<span class="k2">)</span> <span class="k2">{</span> <span class="c">// setting.screen = the allegro display</span>
      <a href="http://www.allegro.cc/manual/al_destroy_display"><span class="a">al_destroy_display</span></a><span class="k2">(</span>setting.screen<span class="k2">)</span><span class="k2">;</span>
      setting.screen <span class="k3">=</span> NULL<span class="k2">;</span>
   <span class="k2">}</span>
<span class="k2">}</span>
</pre></div></div><p>

</p><div class="source-code"><div class="toolbar"><span class="button numbers"><b>#</b></span><span class="button select">Select</span><span class="button expand">Expand</span></div><div class="inner"><span class="number">   1</span><span class="k1">void</span> init_display<span class="k2">(</span><span class="k1">bool</span> reset_display<span class="k2">)</span>
<span class="number">   2</span><span class="k2">{</span>
<span class="number">   3</span>   <span class="k1">char</span> s<span class="k2">[</span>TEXT_BUFFER<span class="k2">]</span> <span class="k3">=</span> <span class="s">""</span><span class="k2">;</span>
<span class="number">   4</span>   <a href="http://www.allegro.cc/manual/ALLEGRO_BITMAP"><span class="a">ALLEGRO_BITMAP</span></a> <span class="k3">*</span>tempBitmap <span class="k3">=</span> NULL<span class="k2">;</span>
<span class="number">   5</span>
<span class="number">   6</span>   <span class="k1">if</span><span class="k2">(</span><span class="k3">!</span>reset_display<span class="k2">)</span> <span class="k2">{</span> <span class="c">// These will already be NULL'd out if this is a reset</span>
<span class="number">   7</span>      <span class="c">// Initialize pointer arrays to zero before anything else so that if we</span>
<span class="number">   8</span>      <span class="c">// have to call shutdown() they can be checked without problems</span>
<span class="number">   9</span>      <a href="http://www.delorie.com/djgpp/doc/libc/libc_569.html" target="_blank">memset</a><span class="k2">(</span>background, <span class="n">0</span>, <span class="k1">sizeof</span><span class="k2">(</span>background<span class="k2">)</span><span class="k2">)</span><span class="k2">;</span>
<span class="number">  10</span>      <a href="http://www.delorie.com/djgpp/doc/libc/libc_569.html" target="_blank">memset</a><span class="k2">(</span>pills, <span class="n">0</span>, <span class="k1">sizeof</span><span class="k2">(</span>pills<span class="k2">)</span><span class="k2">)</span><span class="k2">;</span>
<span class="number">  11</span>      <a href="http://www.delorie.com/djgpp/doc/libc/libc_569.html" target="_blank">memset</a><span class="k2">(</span>lines, <span class="n">0</span>, <span class="k1">sizeof</span><span class="k2">(</span>lines<span class="k2">)</span><span class="k2">)</span><span class="k2">;</span>
<span class="number">  12</span>
<span class="number">  13</span>      setting.screen <span class="k3">=</span> NULL<span class="k2">;</span>
<span class="number">  14</span>      pickup.sheet <span class="k3">=</span> NULL<span class="k2">;</span>
<span class="number">  15</span>      tool.sheet <span class="k3">=</span> NULL<span class="k2">;</span>
<span class="number">  16</span>      shield <span class="k3">=</span> NULL<span class="k2">;</span>
<span class="number">  17</span>      bullet.bitmap <span class="k3">=</span> NULL<span class="k2">;</span>
<span class="number">  18</span>      pacman.sheet <span class="k3">=</span> NULL<span class="k2">;</span>
<span class="number">  19</span>   <span class="k2">}</span>
<span class="number">  20</span>
<span class="number">  21</span>   <span class="c">// Vsync 1 means force on, 2 means forced off.</span>
<span class="number">  22</span>   <span class="k1">if</span><span class="k2">(</span>setting.vsync<span class="k2">)</span>
<span class="number">  23</span>      <a href="http://www.allegro.cc/manual/al_set_new_display_option"><span class="a">al_set_new_display_option</span></a><span class="k2">(</span>ALLEGRO_VSYNC, setting.vsync, ALLEGRO_SUGGEST<span class="k2">)</span><span class="k2">;</span>
<span class="number">  24</span>
<span class="number">  25</span>   <span class="k1">switch</span><span class="k2">(</span>setting.fullscreen<span class="k2">)</span> <span class="k2">{</span>
<span class="number">  26</span>      <span class="k1">case</span> <span class="n">0</span><span class="k2">:</span>  <span class="c">// Windowed</span>
<span class="number">  27</span>         <span class="k1">if</span><span class="k2">(</span>setting.opengl<span class="k2">)</span> <a href="http://www.allegro.cc/manual/al_set_new_display_flags"><span class="a">al_set_new_display_flags</span></a><span class="k2">(</span>ALLEGRO_OPENGL <span class="k3">|</span> ALLEGRO_WINDOWED<span class="k2">)</span><span class="k2">;</span>
<span class="number">  28</span>         <span class="k1">else</span> <a href="http://www.allegro.cc/manual/al_set_new_display_flags"><span class="a">al_set_new_display_flags</span></a><span class="k2">(</span>ALLEGRO_WINDOWED<span class="k2">)</span><span class="k2">;</span>
<span class="number">  29</span>         <span class="k1">break</span><span class="k2">;</span>
<span class="number">  30</span>      <span class="k1">case</span>  <span class="n">1</span><span class="k2">:</span> <span class="c">// Fullscreen Window (0 = window)</span>
<span class="number">  31</span>         <span class="k1">if</span><span class="k2">(</span>setting.opengl<span class="k2">)</span> <a href="http://www.allegro.cc/manual/al_set_new_display_flags"><span class="a">al_set_new_display_flags</span></a><span class="k2">(</span>ALLEGRO_FULLSCREEN_WINDOW <span class="k3">|</span> ALLEGRO_OPENGL<span class="k2">)</span><span class="k2">;</span>
<span class="number">  32</span>         <span class="k1">else</span> <a href="http://www.allegro.cc/manual/al_set_new_display_flags"><span class="a">al_set_new_display_flags</span></a><span class="k2">(</span>ALLEGRO_FULLSCREEN_WINDOW<span class="k2">)</span><span class="k2">;</span>
<span class="number">  33</span>         <span class="k1">if</span><span class="k2">(</span>setting.frequency<span class="k2">)</span> <a href="http://www.allegro.cc/manual/al_set_new_display_refresh_rate"><span class="a">al_set_new_display_refresh_rate</span></a><span class="k2">(</span>setting.frequency<span class="k2">)</span><span class="k2">;</span>
<span class="number">  34</span>         <span class="k1">break</span><span class="k2">;</span>
<span class="number">  35</span>      <span class="k1">case</span> <span class="n">2</span><span class="k2">:</span>  <span class="c">// Regular Fullscreen (0 = window)</span>
<span class="number">  36</span>         <span class="k1">if</span><span class="k2">(</span>setting.opengl<span class="k2">)</span> <a href="http://www.allegro.cc/manual/al_set_new_display_flags"><span class="a">al_set_new_display_flags</span></a><span class="k2">(</span>ALLEGRO_FULLSCREEN <span class="k3">|</span> ALLEGRO_OPENGL<span class="k2">)</span><span class="k2">;</span>
<span class="number">  37</span>         <span class="k1">else</span> <a href="http://www.allegro.cc/manual/al_set_new_display_flags"><span class="a">al_set_new_display_flags</span></a><span class="k2">(</span>ALLEGRO_FULLSCREEN<span class="k2">)</span><span class="k2">;</span>
<span class="number">  38</span>         <span class="k1">if</span><span class="k2">(</span>setting.frequency<span class="k2">)</span> <a href="http://www.allegro.cc/manual/al_set_new_display_refresh_rate"><span class="a">al_set_new_display_refresh_rate</span></a><span class="k2">(</span>setting.frequency<span class="k2">)</span><span class="k2">;</span>
<span class="number">  39</span>         <span class="k1">break</span><span class="k2">;</span>
<span class="number">  40</span>      default: <span class="c">// invalid mode, we'll select windowed by default and reset the fullscreen variable</span>
<span class="number">  41</span>         <span class="k1">if</span><span class="k2">(</span>setting.opengl<span class="k2">)</span> <a href="http://www.allegro.cc/manual/al_set_new_display_flags"><span class="a">al_set_new_display_flags</span></a><span class="k2">(</span>ALLEGRO_OPENGL <span class="k3">|</span> ALLEGRO_WINDOWED<span class="k2">)</span><span class="k2">;</span>
<span class="number">  42</span>         <span class="k1">else</span> <a href="http://www.allegro.cc/manual/al_set_new_display_flags"><span class="a">al_set_new_display_flags</span></a><span class="k2">(</span>ALLEGRO_WINDOWED<span class="k2">)</span><span class="k2">;</span>
<span class="number">  43</span>         setting.fullscreen <span class="k3">=</span> <span class="n">0</span><span class="k2">;</span>
<span class="number">  44</span>   <span class="k2">}</span>
<span class="number">  45</span>
<span class="number">  46</span>   <a href="http://www.allegro.cc/manual/al_set_new_bitmap_flags"><span class="a">al_set_new_bitmap_flags</span></a><span class="k2">(</span>ALLEGRO_MAG_LINEAR<span class="k2">)</span><span class="k2">;</span> <span class="c">// | ALLEGRO_NO_PRESERVE_TEXTURE);</span>
<span class="number">  47</span>   <a href="http://www.allegro.cc/manual/al_set_blender"><span class="a">al_set_blender</span></a><span class="k2">(</span>ALLEGRO_ADD, ALLEGRO_ONE, ALLEGRO_INVERSE_ALPHA<span class="k2">)</span><span class="k2">;</span>
<span class="number">  48</span>
<span class="number">  49</span>   <a href="http://www.allegro.cc/manual/al_set_new_display_option"><span class="a">al_set_new_display_option</span></a><span class="k2">(</span>ALLEGRO_SINGLE_BUFFER, <span class="n">1</span>, ALLEGRO_REQUIRE<span class="k2">)</span><span class="k2">;</span>
<span class="number">  50</span>
<span class="number">  51</span>   <span class="c">// Allegro picks the desktop resolution automatically with ALLEGRO_FULLSCREEN_WINDOW flag set.</span>
<span class="number">  52</span>   setting.screen <span class="k3">=</span> <a href="http://www.allegro.cc/manual/al_create_display"><span class="a">al_create_display</span></a><span class="k2">(</span>WIDTH, HEIGHT<span class="k2">)</span><span class="k2">;</span>
<span class="number">  53</span>   <span class="k1">if</span><span class="k2">(</span><span class="k3">!</span>setting.screen<span class="k2">)</span> <span class="k2">{</span>
<span class="number">  54</span>      a5_error<span class="k2">(</span>AT, setting.screen, <span class="s">"Create display failed."</span><span class="k2">)</span><span class="k2">;</span>
<span class="number">  55</span>      shut_down<span class="k2">(</span><span class="k2">)</span><span class="k2">;</span>
<span class="number">  56</span>      <a href="http://www.delorie.com/djgpp/doc/libc/libc_298.html" target="_blank">exit</a><span class="k2">(</span><span class="n">1</span><span class="k2">)</span><span class="k2">;</span>
<span class="number">  57</span>   <span class="k2">}</span>
<span class="number">  58</span>
<span class="number">  59</span>   setting.w <span class="k3">=</span> <a href="http://www.allegro.cc/manual/al_get_display_width"><span class="a">al_get_display_width</span></a><span class="k2">(</span>setting.screen<span class="k2">)</span><span class="k2">;</span>
<span class="number">  60</span>   setting.h <span class="k3">=</span> <a href="http://www.allegro.cc/manual/al_get_display_height"><span class="a">al_get_display_height</span></a><span class="k2">(</span>setting.screen<span class="k2">)</span><span class="k2">;</span>
<span class="number">  61</span>
<span class="number">  62</span>   <span class="k1">if</span><span class="k2">(</span>setting.opengl<span class="k2">)</span> <a href="http://www.delorie.com/djgpp/doc/libc/libc_624.html" target="_blank">printf</a><span class="k2">(</span><span class="s">"OpenGL version: 0x%08X\n"</span>, <a href="http://www.allegro.cc/manual/al_get_opengl_version"><span class="a">al_get_opengl_version</span></a><span class="k2">(</span><span class="k2">)</span><span class="k2">)</span><span class="k2">;</span>
<span class="number">  63</span>
<span class="number">  64</span>   <span class="c">// Call scale screen for all modes to correct mouse movement in certain mode changes</span>
<span class="number">  65</span>   a5_scale_screen<span class="k2">(</span>WIDTH, HEIGHT, setting.w, setting.h<span class="k2">)</span><span class="k2">;</span>
<span class="number">  66</span>
<span class="number">  67</span>   <a href="http://www.allegro.cc/manual/al_clear_to_color"><span class="a">al_clear_to_color</span></a><span class="k2">(</span><a href="http://www.allegro.cc/manual/al_map_rgb_f"><span class="a">al_map_rgb_f</span></a><span class="k2">(</span><span class="n">0</span>, <span class="n">0</span>, <span class="n">0</span><span class="k2">)</span><span class="k2">)</span><span class="k2">;</span>
<span class="number">  68</span>   <a href="http://www.allegro.cc/manual/al_flip_display"><span class="a">al_flip_display</span></a><span class="k2">(</span><span class="k2">)</span><span class="k2">;</span>
<span class="number">  69</span>
<span class="number">  70</span>   icon <span class="k3">=</span> <a href="http://www.allegro.cc/manual/al_load_bitmap"><span class="a">al_load_bitmap</span></a><span class="k2">(</span><span class="s">"Graphics/Deluxe_Pacman_2_Icon.png"</span><span class="k2">)</span><span class="k2">;</span>
<span class="number">  71</span>   <span class="k1">if</span><span class="k2">(</span><span class="k3">!</span>icon<span class="k2">)</span> <span class="k2">{</span>
<span class="number">  72</span>      a5_error<span class="k2">(</span>AT, setting.screen, <span class="s">"Error loading Deluxe_Pacman_2_Icon.png."</span><span class="k2">)</span><span class="k2">;</span>
<span class="number">  73</span>      shut_down<span class="k2">(</span><span class="k2">)</span><span class="k2">;</span>
<span class="number">  74</span>      <a href="http://www.delorie.com/djgpp/doc/libc/libc_298.html" target="_blank">exit</a><span class="k2">(</span><span class="n">1</span><span class="k2">)</span><span class="k2">;</span>
<span class="number">  75</span>   <span class="k2">}</span>
<span class="number">  76</span>   <a href="http://www.allegro.cc/manual/al_set_display_icon"><span class="a">al_set_display_icon</span></a><span class="k2">(</span>setting.screen, icon<span class="k2">)</span><span class="k2">;</span>
<span class="number">  77</span>
<span class="number">  78</span>   <a href="http://www.allegro.cc/manual/al_set_window_title"><span class="a">al_set_window_title</span></a><span class="k2">(</span>setting.screen, <span class="s">"Deluxe Pacman 2"</span><span class="k2">)</span><span class="k2">;</span>
<span class="number">  79</span>
<span class="number">  80</span>   <span class="c">/// ****** Load Fonts *******</span>
<span class="number">  81</span>   <span class="c">/// ****** Load background tiles *******</span>
<span class="number">  82</span>   <span class="c">/// ****** Load pills *******</span>
<span class="number">  83</span>   <span class="c">/// ******* Load lines *******</span>
<span class="number">  84</span>   <span class="c">/// ******* Load Pickups *******</span>
<span class="number">  85</span>   <span class="c">/// ******* Load Tools *******</span>
<span class="number">  86</span>   <span class="c">/// ******* Load hearts *******</span>
<span class="number">  87</span>   <span class="c">/// ******* Load Ghost Spawn *******</span>
<span class="number">  88</span>   <span class="c">/// ******* Load Pickup Spawn *******</span>
<span class="number">  89</span>   <span class="c">/// ******* Create bitmap for drawing the level on *******</span>
<span class="number">  90</span>   <span class="c">/// ******* Initialize Pacman *******</span>
<span class="number">  91</span>   <span class="c">/// ****** Initialize Ghosts *******</span>
<span class="number">  92</span>
<span class="number">  93</span>   <span class="c">// reset the target bitmap back to the display</span>
<span class="number">  94</span>   <a href="http://www.allegro.cc/manual/al_set_target_bitmap"><span class="a">al_set_target_bitmap</span></a><span class="k2">(</span><a href="http://www.allegro.cc/manual/al_get_backbuffer"><span class="a">al_get_backbuffer</span></a><span class="k2">(</span>setting.screen<span class="k2">)</span><span class="k2">)</span><span class="k2">;</span>
<span class="number">  95</span>
<span class="number">  96</span>   <span class="c">// register the new display in the event_queue</span>
<span class="number">  97</span>   <a href="http://www.allegro.cc/manual/al_register_event_source"><span class="a">al_register_event_source</span></a><span class="k2">(</span>event_queue, <a href="http://www.allegro.cc/manual/al_get_display_event_source"><span class="a">al_get_display_event_source</span></a><span class="k2">(</span>setting.screen<span class="k2">)</span><span class="k2">)</span><span class="k2">;</span>
<span class="number">  98</span><span class="k2">}</span>
</div></div><p>
</p></div>]]>
		</description>
		<author>no-reply@allegro.cc (Neil Roy)</author>
		<pubDate>Thu, 25 May 2017 07:40:54 +0000</pubDate>
	</item>
	<item>
		<description><![CDATA[<div class="mockup v2"><p>Bitmaps that weren&#39;t loaded from files aren&#39;t preserved.  Their contents are erased.</p><p>Also, reloading all bitmaps isn&#39;t realistic when your game reaches over 100 MB ...</p><p>I just saw you update your post.  That is an insane amount of code to resize the display...
</p></div>]]>
		</description>
		<author>no-reply@allegro.cc (roger levy)</author>
		<pubDate>Thu, 25 May 2017 07:54:44 +0000</pubDate>
	</item>
	<item>
		<description><![CDATA[<div class="mockup v2"><div class="quote_container"><div class="title"><a href="http://www.allegro.cc/forums/thread/616905/1030574#target">roger levy</a> said:</div><div class="quote"><p>Also, reloading all bitmaps isn&#39;t realistic when your game reaches over 100 MB ...</p><p>I just saw you update your post. That is an insane amount of code to resize the display... </p></div></div><p>

Try my game out (link in my sig), it works fine, and it works fast.  It&#39;s not like a person is sitting there resizing the display 10 times a second.  They generally resize it ONCE when they first install it and never touch that option again, or rarely.  It&#39;s much faster than you imagine.</p><p>Also, that code is code I use when I initialize the game and destroy it.  I redesigned the functions to easily use them when a screen mode is changed.  No sense re-inventing the wheel.  Works like a charm.</p><p>BUT... if there is a way I can skip reloading bitmaps, I will gladly adopt it.  I never thought to look into it as this works well as is.
</p></div>]]>
		</description>
		<author>no-reply@allegro.cc (Neil Roy)</author>
		<pubDate>Thu, 25 May 2017 08:02:39 +0000</pubDate>
	</item>
	<item>
		<description><![CDATA[<div class="mockup v2"><p>My bad on the bitmap contents being lost.  I was recreating a bitmap as one attempt to stop the crashing.  Took that out and the bitmap contents are unaffected now.
</p></div>]]>
		</description>
		<author>no-reply@allegro.cc (roger levy)</author>
		<pubDate>Thu, 25 May 2017 11:35:33 +0000</pubDate>
	</item>
</rss>
